Skip to content

Rename template info key names in cookiecutter context#1348

Merged
freakboy3742 merged 1 commit intobeeware:mainfrom
rmartin16:cookiecutter-template-fix
Jul 8, 2023
Merged

Rename template info key names in cookiecutter context#1348
freakboy3742 merged 1 commit intobeeware:mainfrom
rmartin16:cookiecutter-template-fix

Conversation

@rmartin16
Copy link
Copy Markdown
Member

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

- In cookiecutter>=2.2.0, the template key in context has special
  meaning for nested templates and must be renamed.
@rmartin16
Copy link
Copy Markdown
Member Author

CI will pass once beeware/briefcase-template#72 is merged; since template is still in cookiecutter.json of the version of briefcase-template that's being used, the error is still occurring.

Copy link
Copy Markdown
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for doing the legwork on this.

@freakboy3742 freakboy3742 merged commit 2dddb34 into beeware:main Jul 8, 2023
@rmartin16 rmartin16 deleted the cookiecutter-template-fix branch July 8, 2023 06:16
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.

Using template as a key in the cookiecutter context has special meaning in versions >=2.2.0

2 participants