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
Remove support for FOSUser bundle #1732
Conversation
b86fbd0
to
4168201
Compare
885cb06
to
4e29f04
Compare
@XWB @stephanvierkant Finally green! :) It does a bit more than expected:
|
05096cb
to
54627cf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice 👍
150f63d
to
5a61706
Compare
We can also clean-up the phpstan config then: https://github.com/hwi/HWIOAuthBundle/blob/master/phpstan.neon#L8 Edit: and also https://github.com/hwi/HWIOAuthBundle/blob/master/.github/workflows/ci.yaml#L145 |
1d8ef79
to
6acc6d1
Compare
6acc6d1
to
1922c82
Compare
@XWB would be great to here your opinion about this, thanks! 🙃 |
I need some time to review, it's a big change. I will make some time in the coming days. |
1922c82
to
c5d290a
Compare
Seems good to me! |
Fixes #1727