-
Notifications
You must be signed in to change notification settings - Fork 184
Symfony3 support #275
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
Symfony3 support #275
Conversation
|
@tgalopin Hi! Are you still working on this PR? |
|
|
||
| matrix: | ||
| allow_failures: | ||
| - env: SYMFONY_VERSION=dev-master |
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.
I don't see the interest of keeping this in the allow failures section as almost no one look at them.
I prefer that the tests break and that we know that we will have to change something.
|
There is still a lot of work here as many part of this bundle are not covered by tests. |
|
Yes, this is very complicated, as the tests are not ready. I'll have to find time to work on the library in order to build something stable for Symfony3. I'm sorry of this :/ . |
|
No problem @tgalopin, you're already doing an awesome job ☺ |
|
@Ener-Getick if we want to adapt this version for Symfony3, it would the best way indeed but I think we should focus the work on the next version that will use the library instead. This version is officially supported for fixes, not for new features (and Symfony3 support is a new feature IMO). |
|
Yes, no problem, I just wanted to know what's up :) FWIW, I started to port stuff to 3.0 so I could migrate one of our project. It's certainly not "the best way to do things", but feel free to have a look and re-use anything you think could help. I've got everything I need working – which is basic features like creating thread and replying. I don't know much about this bundle, so I have no idea how far I've gone in the process of porting the entire thing. |
|
@ubermuda thanks for the work, I'll check that asap :) ! |
|
any news? |
|
When i am trying to add to config.yml and |
|
Oh, just realized @ubermuda has fixed it in his fork. |
|
I'll test this tonight, sorry for the delay. Thanks for your work @ubermuda ! |
|
Added small fix |
|
Just meregd @ubermuda's code into my fork. Branch: https://github.com/ownede/FOSMessageBundle/tree/symfony3 See merge request #283 |
|
Closed in favor of #283 |
No description provided.