Skip to content

Conversation

@lovisaberggren
Copy link
Collaborator

Proposed changes

The reviewers option for dependabot is retired and replaced with codeowners, see blog post announcement

Jira ticket: CLOUDP-327328

@lovisaberggren lovisaberggren marked this pull request as ready for review June 25, 2025 09:45
@lovisaberggren lovisaberggren requested a review from a team as a code owner June 25, 2025 09:45
* @mongodb/apix
* @mongodb/apix
/tools/cli/ @mongodb/apix-2
/tools/spectral/ipa/ @mongodb/apix1
Copy link
Member

@wtrocki wtrocki Jun 25, 2025

Choose a reason for hiding this comment

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

Do we want to postman to be shared between apix1 and apix2?

Copy link
Member

@wtrocki wtrocki Jun 25, 2025

Choose a reason for hiding this comment

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

If yes then:

Suggested change
/tools/spectral/ipa/ @mongodb/apix1
/tools/spectral/ipa/ @mongodb/apix1
/tools/postman/ @mongodb/apix1

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah postman should be apix 1, I've added it to a follow-up 👍

Copy link
Member

@wtrocki wtrocki left a comment

Choose a reason for hiding this comment

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

LGTM

As future consideration/open question I would raise discussion who owns <root>/package.json as this historically driven a lot of dependabot updates

@lovisaberggren
Copy link
Collaborator Author

@wtrocki Yeah there are some things not divided to the specific teams, we could do an improvement as follow-up though apix 1 is going to own all of it soon anyways so I didn't spend any time on it now, I can create a follow-up ticket if we want more granular ownership until next Q 👍

@lovisaberggren lovisaberggren merged commit e7fe8a3 into main Jun 25, 2025
4 checks passed
@lovisaberggren lovisaberggren deleted the CLOUDP-327328 branch June 25, 2025 09:58
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.

3 participants