Skip to content

fix(template): auto updates#11

Merged
JonZeolla merged 4 commits into
mainfrom
fix-update-task
Jul 3, 2025
Merged

fix(template): auto updates#11
JonZeolla merged 4 commits into
mainfrom
fix-update-task

Conversation

@JonZeolla

@JonZeolla JonZeolla commented Jul 3, 2025

Copy link
Copy Markdown
Member

Contributor Comments

This fixes a typo where I had }} instead of {{ and it was causing task update in rendered projects to fail.

It also adds a test to prevent this from reoccurring

I ran the integration tests without the Taskfile.yml fix implemented and saw the expected error:

Screenshot 2025-07-03 at 5 31 58 PM

I also generated a project and tested the workflow dispatch:

Screenshot 2025-07-03 at 5 44 41 PM Screenshot 2025-07-03 at 5 45 03 PM

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

@ai-coding-guardrails ai-coding-guardrails Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I've got 4 comments for you to consider

The PR title and description are not accurate. Here are my suggestions:

Title:

Description:

Are we missing a feature you'd like? Submit it here!

Comment thread tests/test_cookiecutter.py
Comment thread tests/test_cookiecutter.py
@JonZeolla JonZeolla changed the title fix(template): auto update task rendering fix(template): auto updates Jul 3, 2025
@JonZeolla JonZeolla enabled auto-merge (squash) July 3, 2025 21:45
@JonZeolla JonZeolla merged commit d93a822 into main Jul 3, 2025
9 checks passed
@JonZeolla JonZeolla deleted the fix-update-task branch July 4, 2025 12:18
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.

1 participant