Skip to content

fix: vars override failure#467

Merged
msclock merged 1 commit intoserious-scaffold:masterfrom
msclock:sub_copier_vars
Feb 27, 2025
Merged

fix: vars override failure#467
msclock merged 1 commit intoserious-scaffold:masterfrom
msclock:sub_copier_vars

Conversation

@msclock
Copy link
Copy Markdown
Collaborator

@msclock msclock commented Feb 27, 2025

it failed to override vars in copier.yaml. Rules are below:

  • delete field type for non-bool variable.
  • keep field type for bool variable.

it failed to override vars in copier.yaml. Rules are below:
- delete field type for non-bool variable.
- keep field type for bool variable.

Signed-off-by: l.feng <43399351+msclock@users.noreply.github.com>
@msclock msclock enabled auto-merge (squash) February 27, 2025 02:55
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (5d78341) to head (5d1f22f).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #467   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines           28        28           
=========================================
  Hits            28        28           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@msclock msclock merged commit afae29e into serious-scaffold:master Feb 27, 2025
@msclock msclock deleted the sub_copier_vars branch February 27, 2025 03:03
@serious-scaffold
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.6.20 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant