Skip to content
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

Fix for [email protected] failing to create project from template #94

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

emshotton
Copy link
Contributor

Creating a project using this template and [email protected] causes the following error which prevents the project from being created:

🤷   Project Name: test-rs
🔧   Destination: /tmp/test-rs ...
🔧   project-name: test-rs ...
🔧   Generating template ...
✔ 🤷   Which flashing method do you intend to use? · probe-rs
[ 1/17]   Done: .cargo/config.toml
[ 2/17]   Done: .cargo
[ 4/17]   Done: .github/workflows
[ 5/17]   Done: .github
[ 6/17]   Done: .gitignore
[ 7/17]   Done: .vscode/launch.json
[ 8/17]   Done: .vscode/settings.json
[ 9/17]   Done: .vscode
[10/17]   Done: APACHE2.0
[11/17]   Done: Cargo.toml
[12/17]   Done: Embed.toml
[13/17]   Done: MIT
[14/17]   Done: build.rs
[15/17]   Done: memory.x
[16/17]   Done: src/main.rs
[17/17]   Done: src

Error: Substitution skipped, found invalid syntax in
    .github/workflows/ci_checks.yml

Consider adding these files to a `cargo-generate.toml` in the template repo to skip substitution on these files.

Learn more: https://github.com/cargo-generate/cargo-generate#include--exclude.

This change adds .github/workflows/ci_checks.yml to the exclude block in cargo-generate.yaml which fixes the issue.

Note: I'm not sure which version of cargo-generate broke this, but I can confirm it previously worked fine with [email protected]

Thanks for this template! It's been super helpful and saved me a bunch of time.

@emshotton emshotton changed the title Fix for [email protected] failing to create project Fix for [email protected] failing to create project from template Mar 13, 2025
@9names 9names merged commit 863c209 into rp-rs:main Mar 14, 2025
8 checks passed
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.

3 participants