Skip to content

Conversation

lwjohnst86
Copy link
Member

Description

This PR needs a quick review.

Checklist

  • Ran just run-all

@lwjohnst86 lwjohnst86 requested a review from a team as a code owner July 25, 2025 13:21
@lwjohnst86 lwjohnst86 moved this from Todo to In Review in Iteration planning Jul 25, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

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

Isn't the space a jinja convention (I got this from here)? Seems to generate .copier-answers.yml in either case though...

@lwjohnst86
Copy link
Member Author

Yea, a space is fine, but also no space also works 😛 I'd prefer no space.

@lwjohnst86 lwjohnst86 requested a review from martonvago July 25, 2025 14:35
@martonvago
Copy link
Contributor

👍 Then should we remove both spaces though? There's one before as well

@lwjohnst86
Copy link
Member Author

Wow, maybe I need new glasses 😆 I didn't even see that 😆

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 25, 2025
@lwjohnst86 lwjohnst86 merged commit bc50286 into main Jul 25, 2025
5 checks passed
@lwjohnst86 lwjohnst86 deleted the chore/remove-space-from-filename branch July 25, 2025 15:10
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants