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.

Sign up for GitHub

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

Jump to bottom

Laravel 11 migration (storm) #173

Open
wants to merge 114 commits into
base: develop
Choose a base branch
from
Open

Laravel 11 migration (storm) #173

wants to merge 114 commits into from

Conversation

mjauvin
Copy link
Member

@mjauvin mjauvin commented Apr 3, 2024

Continue work started by @bennothommo & @wverhoogt

@mjauvin mjauvin added this to the 1.3.0 milestone Apr 3, 2024
@mjauvin mjauvin self-assigned this Apr 3, 2024
@mjauvin
restore initial larastan package
416c6b3
@mjauvin
adjust $fillable signature
4c7a7af
@mjauvin
adjust store() method return type
d58d314
@mjauvin
fix getCache() return type
3c4013c
@mjauvin
remove invalid getValue() call
f5e4b94
@mjauvin
migrate phpunit xml config
00513c9
@mjauvin
update dependencies for laravel 11
3a86f6c
@mjauvin
allign with new Authenticatable contract
ea41b89
@mjauvin
match symfony SignalableCommandInterface signature
3f5e612
@mjauvin
match new builder paginate signature
9f9dc7a
@mjauvin
update CacheManager signature
da18666
@mjauvin
fix assetic/framework dependency
b18d3e5
@mjauvin
fix mockery dependency
98b0e6a
@mjauvin
replace setMethods() with add/onlyMethods()
7aee659
@mjauvin
fix deprecation errors
79e2747
@mjauvin
we need to extend TestCase now
7b588e2
@mjauvin
call parent constructor
374f00f
@mjauvin
return class instance
16d43fa
@mjauvin
Remove argument removed in Laravel 11
3970819
@mjauvin
PDO driver classes removed in Laravel 11
148ea7c
@mjauvin
try to make EventTest work
d888de7
@mjauvin
remove upload test artifact
5bddeb6
@mjauvin
update php test versions
88f1639
@mjauvin
add comment about deprecated $dates property; remove doctrine/dbal
cccf1b6
@mjauvin
remove unnecessary override of Illuminate\Cache\CacheManager
ec6d8a6
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 2, 2024

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

bennothommo
bennothommo reviewed Oct 7, 2024
View reviewed changes
src/Auth/Manager.php Outdated Show resolved Hide resolved
@mjauvin mjauvin changed the title Laravel 11 migration Laravel 11 migration (storm) Oct 11, 2024
@mjauvin
Copy link
Member Author

mjauvin commented Oct 21, 2024

@LukeTowers @bennothommo @jaxwilko do you guys have any idea what's wrong with those tests ? They worked fine before.

@mjauvin
Copy link
Member Author

mjauvin commented Oct 21, 2024

@LukeTowers @bennothommo @jaxwilko do you guys have any idea what's wrong with those tests ? They worked fine before.

Laravel made some upstream changes that broke the tests.
All good now.

@mjauvin
Copy link
Member Author

mjauvin commented Oct 21, 2024

Ready to merge in develop when 1.2.7 gets released! 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bennothommo bennothommo bennothommo left review comments

@LukeTowers LukeTowers LukeTowers left review comments

Assignees

@mjauvin mjauvin

Labels
None yet
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.
4 participants
@mjauvin @LukeTowers @bennothommo @wverhoogt