-
Notifications
You must be signed in to change notification settings - Fork 93
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
Jinja2.10 + handle StandardError from processing #2553
Conversation
#------------------------------------------------------------------------------- | ||
purge_suite $SUITE_NAME | ||
|
||
cat >'suite.rc' <<'__SUITERC__' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete lines 44-51?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
d9265a3
to
6130223
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good, but just need to update the Jinja2 version mentioned under "Software Packages Bundled with Cylc" in the User Guide (installation section), and section 9.7.5 (Jinja2 Variable Scope) needs to be updated to show how to use the new namespace feature to access out-of-scope variables (actually our existing example has an error in it too - COUNT vs FOO)
Version and scope advice.
(new commit reviewed and approved) |
Close #2497. Fix #2550.