Skip to content

Conversation

@lwjohnst86
Copy link
Member

Description

These changes add some commands to run right after copying from the template.

Closes #95

This PR needs a quick review.

Checklist

  • Formatted Markdown
  • Ran just run-all

@lwjohnst86 lwjohnst86 moved this from Todo to In Review in Iteration planning Jul 24, 2025
Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Some suggestions 💡 :

copier.yaml Outdated
# Post-copy commands:
_tasks:
# Only run these commands when copying the template
- command: "git init"
Copy link
Member

Choose a reason for hiding this comment

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

I'm guessing that you can't turn this around to when "{{ _copier_operation == 'copy' }}", then run all commands?

Copy link
Member Author

Choose a reason for hiding this comment

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

I couldn't see a way that I could run multiple commands. I guess I could use something like &&.

martonvago
martonvago previously approved these changes Jul 24, 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.

One suggestion, but I approve 👍

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Jul 24, 2025
@lwjohnst86 lwjohnst86 merged commit ef57d74 into main Jul 24, 2025
4 of 5 checks passed
@lwjohnst86 lwjohnst86 deleted the feat/add-post-copy-commands branch July 24, 2025 17:40
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Jul 24, 2025
lwjohnst86 added a commit that referenced this pull request Jul 24, 2025
# Description

Runs the justfile checks from the built data package created from the
template.

This depends on #101 and doesn't work without.

Closes #56 

This PR needs a quick review.

## Checklist

- [ ] Ran `just run-all`
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.

Add post-copy steps to tasks

4 participants