Skip to content

Moved dependabot reviewers to codeowners #17745

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

oakrizan
Copy link

Release notes

[rn:skip]

What does this PR do?

Removed reviewers section in dependabot.yml and added CODEOWNERS.yml with related definition for observablt-ci && observablt-ci-contractors

Why is it important/What is the impact to the user?

Reviewers dependabot.yml configuration is being retired option because the functionality overlaps with GitHub code owners.
See: https://github.blog/changelog/2025-04-29-dependabot-reviewers-configuration-option-being-replaced-by-code-owners/

Checklist

  • [x ] My code follows the style guidelines of this project
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have made corresponding change to the default configuration files (and/or docker env variables)
  • [ ] I have added tests that prove my fix is effective or that my feature works

Related issues

Logs

@oakrizan oakrizan added enhancement ci backport-active-all Automated backport with mergify to all the active branches labels Jun 26, 2025
@oakrizan oakrizan requested a review from v1v June 26, 2025 10:45
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@oakrizan oakrizan requested a review from dliappis June 26, 2025 10:45
@oakrizan oakrizan linked an issue Jun 26, 2025 that may be closed by this pull request
Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

LGTM - however, it's better to confirm if the CODEOWNERS won't cause any harm here with the branch protections

@oakrizan
Copy link
Author

buildkite/logstash-exhaustive-tests-pipeline failure is not related to the changes in this PR, it fails on main as well

@donoghuc
Copy link
Member

yeah, the exhaustive test failures are not related to this (missing some builds on the newly bumped 9.2.0 version)

@oakrizan
Copy link
Author

oakrizan commented Jul 3, 2025

@dliappis, PTAL

Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

Lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-active-all Automated backport with mergify to all the active branches ci enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: move from dependabot reviewers to CODEOWNERS
4 participants