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.

Sign up for GitHub

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

Jump to bottom

Add jinja env docs #1872

Merged
merged 6 commits into from Jul 1, 2023
Merged

Add jinja env docs #1872

merged 6 commits into from Jul 1, 2023

Conversation

pamelafox
Copy link
Contributor

This PR documents _jinja2_env_vars, which was added in 2020.

CC @kjaymiller who just used it in a project

kurtmckee
kurtmckee suggested changes Jun 29, 2023
View reviewed changes
Copy link
Member

@kurtmckee kurtmckee left a comment

Choose a reason for hiding this comment

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

Thanks so much for this work! I've requested a change to the Jinja documentation, but otherwise this looks good to me!

@pydanny @jensens @ericof Please merge this PR after the Jinja documentation link is updated. Thanks!

docs/advanced/jinja_env.rst Outdated Show resolved Hide resolved
@pamelafox @kurtmckee
Update docs/advanced/jinja_env.rst
69f7ec5 
Co-authored-by: Kurt McKee <contactme@kurtmckee.org>
@pamelafox
Copy link
Contributor Author

@kurtmckee Thanks, great catch!

@pamelafox
Copy link
Contributor Author

Ah I bet the checks failed due to GitHub outage this morning, as they passed previously.

@ericof ericof added the documentation This issue/PR relates to or includes documentation. label Jul 1, 2023
@ericof ericof added this to the 2.2.0 milestone Jul 1, 2023
@ericof ericof merged commit 982940a into cookiecutter:main Jul 1, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kurtmckee kurtmckee kurtmckee approved these changes

@ericof ericof ericof approved these changes

Assignees
No one assigned
Labels
documentation This issue/PR relates to or includes documentation.
Projects
None yet
Milestone
2.2.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pamelafox @kurtmckee @ericof @jensens