Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: Update code-owner-self-merge ref #3592

Conversation

hyperupcall
Copy link
Member

@hyperupcall hyperupcall commented Feb 16, 2024

To reduce maintenance burden and custom CI code, I'm upstreaming the ownerNoPings option that I added to my code-owner-self-merge fork. I'm switching the ref to the specific branch that'll serve as the basis for the upstream PR. Doing this will "test" the code.

This is tracked upstream at OSS-Docs-Tools/code-owner-self-merge#42.

Once this is merged upstream, then I'll make another PR that updates the same workflow to use the upstream project instead of my fork.

@github-actions github-actions bot added ci ".github/**/*" folder is updated (auto-generated by labeler action) NodeJS NodeJS files are updated (auto-generated by labeler action) labels Feb 16, 2024
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ci ".github/**/*" folder is updated (auto-generated by labeler action) NodeJS NodeJS files are updated (auto-generated by labeler action)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant