Skip to content

Conversation

@NicholasEllul
Copy link
Contributor

@NicholasEllul NicholasEllul commented Apr 10, 2024

Explanation

This removes the deprecated metamask/devs CODEOWNER with metamask/engineering. This is due to the devs team being deprecated.

References

Changelog

None

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@NicholasEllul NicholasEllul requested a review from a team as a code owner April 10, 2024 21:02
@NicholasEllul NicholasEllul changed the title Ellul/codeowner remove devs Remove explicit metamask/devs CODEOWNER Apr 10, 2024
@NicholasEllul NicholasEllul force-pushed the ellul/codeowner-remove-devs branch from e0fcd75 to 2689213 Compare April 11, 2024 15:58
@NicholasEllul NicholasEllul changed the title Remove explicit metamask/devs CODEOWNER Replace MetaMask/Devs with MetaMask/Engineering CODEOWNER Apr 11, 2024
Copy link
Contributor

@desi desi left a comment

Choose a reason for hiding this comment

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

LGTM

@NicholasEllul NicholasEllul merged commit 4bf3498 into main Apr 17, 2024
@NicholasEllul NicholasEllul deleted the ellul/codeowner-remove-devs branch April 17, 2024 17:38
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