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

[Resource] Fix broken spec in ResourceBundle #5045

Merged

Conversation

Zales0123
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Related tickets
License MIT

It should be deeply investigated, why this nasty bug has not been caught by Travis build.

@Zales0123 Zales0123 changed the title [Resource] Fix broken spec in *ResourceBundle* [Resource] Fix broken spec in ResourceBundle May 17, 2016
@michalmarcinkowski michalmarcinkowski merged commit e90244d into Sylius:master May 17, 2016
@michalmarcinkowski
Copy link
Contributor

Thanks Mateusz! 👍

Travis and the build is fine. The broken build on master was caused by merging the outdated PR (#4618). We could prevent that by enforcing the PRs to be always rebased with the newest master, but this will require to rebase all PRs after each merge, so for practical reasons we must live with this happening from time to time 😉

@Zales0123 Zales0123 deleted the fix-broken-spec-on-master branch October 6, 2016 20:28
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

2 participants