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

Switch to Laravel 8.37 >= anonymous migrations #961

Merged
merged 1 commit into from
Jan 22, 2022

Conversation

geisi
Copy link
Contributor

@geisi geisi commented Jan 21, 2022

Hello,

this PR refactors all of the migrations to anonymous migrations.

Since the upcoming Laravel 9 release is pretty close maybe it is time to drop some old dependencies (Laravel 8 & PHP 7.x) in the Jetstream package and start doing changes for a next major Jetstream (3.x) version?

So this is a kind of a tryout pr. I just want to checkout how the future of Jetstream Open Source development will look like. And how these kind of PRs are received by the maintainers.

I'm not sure how the Jetstream Package Major Versions release cycle is going to look like in the future since this is the first Major Laravel Release after Jetstream was born ( it really feels like a long time ago).

I hope I'm not annoying anyone with this PR. Feel free to close this pr and thank you for your feedback!

Have a nice weekend!

@driesvints
Copy link
Member

This can be sent to 2.x I think since there's no breaking changes. We atm have no plans for Jetstream v3

@geisi geisi changed the base branch from master to 2.x January 21, 2022 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants