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

Proposal for fixing custom bootstrap.less generator #301

Conversation

mumia
Copy link
Contributor

@mumia mumia commented Sep 11, 2014

Refers #300

This works well in SF 2.5.4.

@florianeckerstorfer
Copy link
Member

Does this also work fine in Symfony < 2.5?

@florianeckerstorfer
Copy link
Member

@mumia Did you take a look at the failed test results? There seems to be a problem.

@mumia
Copy link
Contributor Author

mumia commented Sep 16, 2014

@florianeckerstorfer no, I was in a hurry to finish the code I was working on.
But I will see what I broke and make sure it is fixed.

@florianeckerstorfer
Copy link
Member

As far as I can see you just forgot to adapt the tests.

@mumia mumia force-pushed the issue300/generatorFailsForCustomVariable.less branch from d7d3147 to e7e4321 Compare September 17, 2014 21:28
@mumia
Copy link
Contributor Author

mumia commented Sep 17, 2014

@florianeckerstorfer Updated the tests and added some protection for older versions of symfony.
Should be ok now.

florianeckerstorfer pushed a commit that referenced this pull request Sep 18, 2014
…riable.less

Proposal for fixing custom bootstrap.less generator
@florianeckerstorfer florianeckerstorfer merged commit f4ba62a into braincrafted:develop Sep 18, 2014
@florianeckerstorfer florianeckerstorfer added this to the 2.1.1 milestone Nov 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants