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

Add support for boolean globals #694

Merged
merged 1 commit into from
Mar 10, 2016
Merged

Conversation

mariusbuescher
Copy link
Contributor

When calling the getGlobal function it throws an error when the global you want to get is falsey. It should only throw an error when the global is undefined sou you can use real boolean globals in a function (e.g. a filter).

Only throw an error when a global is "undefined" when calling
getGlobal function in environment.
carljm added a commit that referenced this pull request Mar 10, 2016
Add support for boolean globals
@carljm carljm merged commit 9675ec2 into mozilla:master Mar 10, 2016
carljm added a commit that referenced this pull request Mar 10, 2016
Add support for boolean globals
carljm added a commit that referenced this pull request Mar 10, 2016
* 2.x: (43 commits)
  Bump versions for dev.
  Update maintenance docs.
  Revert accidental changes to mocha.js.
  Bump version to 2.4.0.
  Update changelog.
  Merge pull request #694 from mariusbuescher/master
  Add note about include and blocks; update wrapping of templating docs.
  Merge pull request #688 from pra85/patch-1
  Rename all test templates to use .j2 extension.
  Update changelog.
  Revert "make include statements trigger async conversion inside if/for statements (fixes #372)"
  Merge pull request #672 from TrySound/yargs
  Update changelog.
  Split backported and non-backported portions of #667 in changelog.
  More acks in changelog.
  Merge pull request #668 from oyyd/cn-doc
  Don't allow included templates to write to the including scope.
  Update changelog for opts.dev fix.
  Split include tests.
  Fix references to env dev opt.
  ...
carljm added a commit that referenced this pull request Mar 14, 2016
* master: (88 commits)
  Bump versions for dev.
  Update maintenance docs.
  Revert accidental changes to mocha.js.
  Bump version to 2.4.0.
  Update changelog.
  Merge pull request #694 from mariusbuescher/master
  Update changelog.
  Add support for boolean globals
  Add note about include and blocks; update wrapping of templating docs.
  Merge pull request #688 from pra85/patch-1
  Add note about include and blocks; update wrapping of templating docs.
  Update api.md
  fixed bad character leading % in {% endraw %}
  Respect null/none as a value in its own right, distinct from undefined. Fixes #478.
  Use dot reporter for npm test.
  Don't bail on first failed test in 'npm test'.
  Add acknowledgement for #561 to changelog.
  Tighter scoping of vars in blocks, to match Jinja2. Fixes #561.
  Rename all test templates to use .j2 extension.
  Rename all test templates to use .j2 extension.
  ...
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