Skip to content

chore: add CODEOWNERS for code-owner review enforcement#8

Merged
eitanp461 merged 4 commits into
masterfrom
chore/add-codeowners
Jun 7, 2026
Merged

chore: add CODEOWNERS for code-owner review enforcement#8
eitanp461 merged 4 commits into
masterfrom
chore/add-codeowners

Conversation

@eitanp461

Copy link
Copy Markdown
Contributor

Summary

Adds a .github/CODEOWNERS file so the branch ruleset's "Require review from Code Owners" check has owners to enforce, strengthening protection on master.

Changes

  • Add .github/CODEOWNERS with @eitanp461 and @sveta-slepner as default owners for the entire repo (single * rule — both listed on one line so neither overrides the other)

Notes

  • Takes effect only once merged to master (GitHub reads CODEOWNERS from the PR base branch) and with Require review from Code Owners enabled in the ruleset

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@sveta-slepner

sveta-slepner commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

It's stupid annoying, but you need to bump the version of the package. Otherwise when this one will be published as well and cause a bug later on N8N UI can't figure which hash is the actual version and the "update" button won't go away :{

eitanp461 and others added 2 commits June 7, 2026 11:42
Every merge to master publishes; 0.1.1 is taken by PR #9.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
# Conflicts:
#	package-lock.json
#	package.json

@sveta-slepner sveta-slepner left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🔥

@eitanp461 eitanp461 merged commit 70f25ea into master Jun 7, 2026
1 check passed
@eitanp461 eitanp461 deleted the chore/add-codeowners branch June 7, 2026 08:54
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