Skip to content

fix init.rhai by adding quote marks#24

Merged
petekubiak merged 1 commit intodysonltd:mainfrom
tom-flaherty:bugfix/fix-cargo-init-fail-due-to-initrhai
Dec 8, 2025
Merged

fix init.rhai by adding quote marks#24
petekubiak merged 1 commit intodysonltd:mainfrom
tom-flaherty:bugfix/fix-cargo-init-fail-due-to-initrhai

Conversation

@tom-flaherty
Copy link
Contributor

@tom-flaherty tom-flaherty commented Dec 8, 2025

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

Please Tick the options that are relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Create a Rust project
esp-generate
Add the modified version of commitment issues
cargo add --git https://github.com/tom-flaherty/commitment-issues
Use cargo generate with the modified repo. There script should ask questions about the repo, so there should not be any message such as 'Failed executing script: init.rhai'
cargo generate -g https://github.com/tom-flaherty/commitment-issues --init

Your Specific Test Case

Checklist

  • My code follows the style guidelines of this project (this should be caught by the CI)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings (this should be caught by the CI)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@petekubiak petekubiak merged commit 80b762a into dysonltd:main Dec 8, 2025
7 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