Skip to content

fix(publish): remove --apply, add summary, fix version commit#80

Merged
MisaelMa merged 4 commits into
devfrom
fix/publish-commit-and-summary
Apr 7, 2026
Merged

fix(publish): remove --apply, add summary, fix version commit#80
MisaelMa merged 4 commits into
devfrom
fix/publish-commit-and-summary

Conversation

@MisaelMa
Copy link
Copy Markdown
Owner

@MisaelMa MisaelMa commented Apr 7, 2026

Summary

  • Remove --apply flag from prerelease rush publish — it was reverting file changes after publish, preventing version bumps from being committed
  • Add publish summary at end of loop: total scopes, published count, failed count
  • Fix commit step: add git add -A before checking for staged changes
  • Keep git add -A after each successful publish to protect changes from git checkout -- . on failures

Fixes

  • Version bumps now persist after publish (committed + pushed to branch)
  • github-release.js can now detect bumped packages via git diff
  • CHANGELOG.json entries will be generated correctly

Test plan

  • Verify publish summary appears in Actions log
  • Verify version bumps are committed and pushed to dev
  • Verify GitHub Releases are created

@MisaelMa MisaelMa added the xml cfdi-xml label Apr 7, 2026
@MisaelMa MisaelMa merged commit edc1289 into dev Apr 7, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

xml cfdi-xml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant