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

Use none instead of None. #322

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

freyes
Copy link
Member

@freyes freyes commented Feb 15, 2024

Jinja2 tests against none (lowercase). When using None (capitalized), the following error is raise:

The conditional check '...' failed. The error was: template error while templating string: no test named 'None'.

See pallets/jinja#520 for more details.

Issue seen in the gate at
https://openstack-ci-reports.ubuntu.com/artifacts/b6d/908775/3/check/jammy-zed/b6df748/job-output.txt

Jinja2 tests against `none` (lowercase). When using `None`
(capitalized), the following error is raise:

The conditional check '...' failed. The error was: template error while templating string: no test named 'None'.

See pallets/jinja#520 for more details.

Issue seen in the gate at
https://openstack-ci-reports.ubuntu.com/artifacts/b6d/908775/3/check/jammy-zed/b6df748/job-output.txt
@freyes freyes requested a review from wolsen February 15, 2024 19:44
@wolsen wolsen merged commit c84cb32 into openstack-charmers:master Feb 15, 2024
2 checks passed
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.

2 participants