-
Notifications
You must be signed in to change notification settings - Fork 0
ci: 👷 add workflow to test template creation #63
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.
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
Conversation
…edcase-project/template-data-package into ci/test-copier-template-build
.github/workflows/test.yml
Outdated
| uvx copier copy --vcs-ref=HEAD ${{ env.TEMPLATE_NAME }} test-data-package \ | ||
| --defaults \ | ||
| --data project_name="My Project" \ | ||
| --data project_description="." \ | ||
| --data project_url="https://github.com/user/repo" \ | ||
| --data author_name="John Smith" \ | ||
| --data author_email="john@example.com" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't work yet. Plus, it will need to be updated after we've converted things and added the variables in copier.yaml.
…ata-package into ci/test-copier-template-build
…edcase-project/template-data-package into ci/test-copier-template-build
| run: sudo apt-get install -y just | ||
|
|
||
| - name: Test and check template creation | ||
| run: just test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This depends on #69 now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since #69 have been merged now, I guess this should be fine 👌
Description
This workflow tests that the (future) template gets created properly and also runs checks on the generated project. I added some TODO items since we'll probably have to do updates once we actually have the template setup.
Closes #35
This PR needs a quick review.