Skip to content

Conversation

rrabenda
Copy link
Contributor

@rrabenda rrabenda commented Oct 4, 2024

Explanation

CI pipeline for auto generated issues when major release of any package occurs. New issues will be created in metamask-extension and metamask-mobile repositories.

References

Changelog

None

@rrabenda rrabenda requested a review from a team as a code owner October 4, 2024 11:37
@rrabenda rrabenda force-pushed the feat/INFRA-2087-auto-generate-issues branch from 8e95e76 to dc8c828 Compare October 4, 2024 11:57
Gudahtt
Gudahtt previously requested changes Oct 4, 2024
@rrabenda rrabenda requested review from a team and Gudahtt October 10, 2024 12:05
Copy link
Member

Choose a reason for hiding this comment

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

Thoughts on mapping each controller to an array of team labels, rather than just one? Many of them have multiple code owners. Choosing just one for the label seems a bit arbitrary.

Copy link
Member

Choose a reason for hiding this comment

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

Done in ad9b711

makemesteaks
makemesteaks previously approved these changes Oct 25, 2024
@Gudahtt Gudahtt force-pushed the feat/INFRA-2087-auto-generate-issues branch from c1d6100 to 5dce762 Compare November 1, 2024 15:01
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM! Though I'll find another reviewer, since I've committed directly to this branch.

@Gudahtt Gudahtt dismissed their stale review November 1, 2024 15:02

Changes have been addressed

@Gudahtt Gudahtt requested a review from makemesteaks November 1, 2024 15:02
Copy link
Contributor

@cryptodev-2s cryptodev-2s left a comment

Choose a reason for hiding this comment

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

Looks great! Excited to see this feature go live.

Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Looks good!

@Gudahtt Gudahtt merged commit 5526c7a into main Nov 1, 2024
120 checks passed
@Gudahtt Gudahtt deleted the feat/INFRA-2087-auto-generate-issues branch November 1, 2024 15:38
Gudahtt added a commit that referenced this pull request Nov 1, 2024
## Explanation

The `create-update-issues` workflow was failing because it didn't have
access to the secrets it needed. The `CORE_CREATE_UPDATE_ISSUES_TOKEN`
secret is only available on hte `default-branch` environment.

The workflow has been updated to use that environment so that it may
access the secret.

Example failure:
https://github.com/MetaMask/core/actions/runs/11633589453/job/32399521142

## References

Fixes bug introduced in #4763

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
- [x] I've prepared draft pull requests for clients and consumer
packages to resolve any breaking changes
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.

5 participants