-
Notifications
You must be signed in to change notification settings - Fork 74
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
Make compatible with Symfony 4 #98
Comments
Hi @cilefen i've updated the googledocs, i'm going to work on it this week i'll keep you posted :) |
@acidjames I am available to help test against SF4 when ready. |
@cilefen thanks, let's try both then, better be more than alone ! I'll try to update about my tests this coming week see you, and have fun ! |
Progress can be seen here : https://github.com/acidjames/BeSimpleSsoAuthBundle https://travis-ci.org/acidjames/BeSimpleSsoAuthBundle Tests are running but failing ATM, you can use my .travis.yml, Makefile You also need to reformat Resources/config .yml files if you want tests to run |
@cilefen Hi, any advances ? Seems like this project needs a lot of rework to make tests successful in Travis, what do you think ? (dependencies not met) |
@acidjames Sorry, I've nothing to add right now. Indeed this is true. Actually, this needs work in order to pass tests at all, as I couldn't get phpunit to pass locally. |
@acidjames There is a tracking sheet: |
@acidjames Oh some of this is my ignorance of bundle testing. |
@cilefen me too ... don't worry |
@acidjames See also PRayno/CasAuthBundle#11 |
@cilefen this is another bundle right ? For users looking for an alternative ? |
@acidjames Yes. It uses Symfony Guard and is simpler. |
See http://symfony.com/blog/helping-prepare-for-symfony-4-bundle-support
The text was updated successfully, but these errors were encountered: