Skip to content

chore: share release file ownership with serverless-aws - #759

Merged
ava-silver merged 2 commits into
mainfrom
ava.silver/chore/add-serverless-aws-codeowners
Nov 6, 2026
Merged

ava-silver merged 2 commits into
mainfrom
ava.silver/chore/add-serverless-aws-codeowners

Conversation

@ava-silver

@ava-silver ava-silver commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds @DataDog/serverless-aws as a code owner of the files required to release the plugin:

  • package.json
  • src/layers.json
  • src/layers-gov.json

Motivation

Releasing the plugin requires a code owner approval, and @DataDog/serverless-onboarding-and-enablement is currently the only owner. The Serverless AWS team can't merge the release PRs (layer catalog updates, version bumps) without help.

Both teams now appear on the same CODEOWNERS line for these files, so an approval from either team satisfies the requirement.

Testing Guidelines

None beyond CI. CODEOWNERS takes effect only after the change lands on the default branch, so the new ownership applies to subsequent release PRs.

Additional Notes

  • CODEOWNERS uses last-match-wins, and GitHub requires one code owner approval per file. Both teams must be on the same line for either approval to count.
  • src/runtime-catalog.json was part of some earlier layer catalog updates but not recent ones (for example, #755). Add it here if future catalog updates touch it.

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ava-silver
ava-silver marked this pull request as ready for review November 6, 2026 16:11
@ava-silver
ava-silver requested a review from a team as a code owner November 6, 2026 16:11
@ava-silver
ava-silver merged commit 192e7b9 into main Nov 6, 2026
11 checks passed
@ava-silver
ava-silver deleted the ava.silver/chore/add-serverless-aws-codeowners branch November 6, 2026 16:32
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