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

Adds set block assignments a la jinja2 #656

Merged
merged 4 commits into from
Feb 2, 2016

Conversation

daniele-rapagnani
Copy link
Contributor

I've implemented set block assignments as requested in #582.
I've updated the docs accordingly, taking inspiration by the jinja documentation.

@@ -1,4 +1,4 @@
/*! Browser bundle of nunjucks 2.3.0 (slim, only works with precompiled templates) */
/*! Browser bundle of nunjucks 3.0.0-dev.1 (slim, only works with precompiled templates) */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert the changes to the browserfiles; that's done only at release time, otherwise it causes too many conflicts.

@carljm
Copy link
Contributor

carljm commented Feb 1, 2016

Very nicely done, thanks so much for the PR! A couple minor comments above, ping me when they're addressed and I'll press the button!

@daniele-rapagnani
Copy link
Contributor Author

Ok, that should do it!
Thanks for the corrections, let me know if I missed anything else.

carljm added a commit that referenced this pull request Feb 2, 2016
Adds set block assignments a la jinja2
@carljm carljm merged commit 65bd48b into mozilla:master Feb 2, 2016
carljm added a commit that referenced this pull request Feb 2, 2016
Adds set block assignments a la jinja2
@carljm carljm mentioned this pull request Feb 2, 2016
@carljm carljm mentioned this pull request Mar 10, 2016
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