Skip to content

Conversation

@PatrickSys
Copy link
Owner

What

  • Remove explicit CODEOWNERS + drop code-owner-review requirement
  • Use release-please manifest config (avoid invalid inputs + keep tags as vX.Y.Z)
  • Make publish workflow idempotent (skip publish if version already exists on npm)

Why

  • Require reviews for external contributors without forcing a named CODEOWNER
  • Unblock release-please PR creation (and avoid accidental version rollback)
  • Allow seeding tags/releases safely

Remove CODEOWNERS requirement, switch to manifest config, and harden npm publish against already-published versions.
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@PatrickSys PatrickSys merged commit e7737e9 into master Jan 28, 2026
3 checks passed
@PatrickSys PatrickSys deleted the chore/release-flow-fixes branch January 28, 2026 19:05
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.

2 participants