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

test definition of global macros from within a simple include file #465

Closed
wants to merge 2 commits into from

Conversation

boutell
Copy link

@boutell boutell commented Jun 22, 2015

This test currently fails with this error:

Error: Unable to call foo, which is undefined or falsey

@garygreen
Copy link
Contributor

This doesn't work in jinja and I don't think we will support it. If you want to use macros you should import them.

@garygreen garygreen closed this Jul 5, 2015
@boutell
Copy link
Author

boutell commented Jul 5, 2015

... Oops. Apparently we've been doing it wrong from day one, then. But the
jinja docs support what you're saying, yes. We'll just have to go modern
before we can go 2.0.

On Sun, Jul 5, 2015 at 10:04 AM, Gary Green [email protected]
wrote:

Closed #465 #465.


Reply to this email directly or view it on GitHub
#465 (comment).

*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE | (215) 755-1330 | punkave.com

@ArmorDarks
Copy link

I'm a bit confused. So, does Jinja support it, or not?

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

3 participants