Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Laravel 10 #14

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Laravel 10 #14

merged 1 commit into from
Oct 8, 2024

Conversation

albinhallen
Copy link
Contributor

Todo

Create new branch v2 where this should merge to

@albinhallen albinhallen requested a review from a team as a code owner October 7, 2024 08:29
@albinhallen albinhallen changed the base branch from main to v2.x October 7, 2024 08:35
Copy link
Collaborator

@kickapoo kickapoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah!

@albinhallen albinhallen added this to the cooldown_39 milestone Oct 7, 2024
@albinhallen
Copy link
Contributor Author

Tested locally and can verify the following works:

  • Subject
  • Body
  • Attachments/uploads

Unverified stuff:

  • How is it supposed to look like? Need to compare with another version

- name: Static analysis
run: vendor/bin/phpstan --memory-limit=512M analyze

- name: Migrate phpunit schema
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When running dependency-version: [prefer-lowest, prefer-stable], the versions don't use the same schema so need to migrate it before running

@albinhallen
Copy link
Contributor Author

Tests

Sent from lime.lime-forms.com

image

Sent on local with this branch (big header image...)

image

@albinhallen
Copy link
Contributor Author

Based on this guide on maintenance branches, we should create the 1.x branch before we merge to main?

https://semantic-release.gitbook.io/semantic-release/usage/workflow-configuration#maintenance-branches

@albinhallen albinhallen force-pushed the laravel-10 branch 5 times, most recently from 441134e to e14bcb4 Compare October 8, 2024 08:19
adamczykpiotr
adamczykpiotr previously approved these changes Oct 8, 2024
BREAKING CHANGE: Requires Laravel v10 or above

fix: phpstan ignore mocks

fix: add missing package

test: add test with attachments included

ci: only run supported versions of php

chore: clean up

fix: change method to set attachment headers

ci: change phpunit schema in runtime

fix: fix typing

fix: include phpunit and phpstan in composer.json

ci: update workflow

build: basic docker compose for local development

feat: Laravel 10 compatability
@albinhallen albinhallen changed the base branch from 2.x to main October 8, 2024 08:24
@albinhallen albinhallen dismissed adamczykpiotr’s stale review October 8, 2024 08:24

The base branch was changed.

Copy link

@adamczykpiotr adamczykpiotr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job!

@adamczykpiotr adamczykpiotr merged commit 9084df7 into main Oct 8, 2024
6 checks passed
@adamczykpiotr adamczykpiotr deleted the laravel-10 branch October 8, 2024 11:11
@lime-ci
Copy link
Collaborator

lime-ci commented Oct 8, 2024

🎉 This PR is included in version 2.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants