Skip to content

ci: fix release-please versioning config key - #261

Merged
xiaoyijun merged 1 commit into
masterfrom
xiaoyijun-ci-fix-versioning-key
Jun 11, 2026
Merged

ci: fix release-please versioning config key#261
xiaoyijun merged 1 commit into
masterfrom
xiaoyijun-ci-fix-versioning-key

Conversation

@xiaoyijun

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #257, which had no effect: the manifest config key for the versioning strategy is versioning, not versioning-strategy (see manifest.tsversioning: config['versioning']; unknown keys are silently ignored). That's why the post-merge run logged PR #245 remained the same instead of refreshing it to 3.0.0-beta.

With the correct key, the merge push of this PR will trigger release-please to recompute and retitle #245 to release: 3.0.0-beta.

Testing

N/A (verified against release-please source and the run log; will be exercised by the merge push of this PR)

Checklist

  • .changeset (N/A — this repo uses release-please)
  • unit tests (N/A)
  • integration tests (N/A)
  • necessary KDoc comments (N/A)

🤖 Generated with Claude Code

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the Release Please configuration so the prerelease versioning strategy is actually applied (Release Please reads versioning, and silently ignores unknown keys like versioning-strategy), enabling the release PR to refresh to 3.0.0-beta as intended.

Changes:

  • Replace the incorrect versioning-strategy config key with the correct versioning key in release-please-config.json.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xiaoyijun
xiaoyijun merged commit 3bc6379 into master Jun 11, 2026
4 checks passed
@xiaoyijun
xiaoyijun deleted the xiaoyijun-ci-fix-versioning-key branch June 11, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants