Skip to content

Conversation

@rekmarks
Copy link
Member

Removes the default approval controller type, and requires an explicitly specified type for all approvals.

@rekmarks rekmarks requested a review from Gudahtt December 17, 2020 08:08
@rekmarks rekmarks requested a review from a team as a code owner December 17, 2020 08:08
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!

@rekmarks rekmarks merged commit 8a914d1 into develop Jan 19, 2021
@rekmarks rekmarks deleted the approval-controller-remove-default-type branch January 19, 2021 02:01
Mrtenz pushed a commit that referenced this pull request Oct 16, 2025
This PR does the following:
- bumps `action-create-release-pr` to `v3` (May fix
https://github.com/MetaMask/create-release-branch/actions/runs/14524263811)

- formats `CHANGELOG.md` using Prettier and updates the pipelines to use
the correct parameters for changelog validation: both non-RC `yarn
auto-changelog validate --prettier` and RC `yarn auto-changelog validate
--rc --prettier` branches.
Mrtenz pushed a commit that referenced this pull request Oct 16, 2025
There seems to be a regression in Node 22.5.0 which [prevents `yarn
install` from running][1] and in turns prevents CI from completing
successfully. This regression was [fixed in 22.5.1][2]. We are using
`22.x` in CI, so in theory it should be using this version, but that
does not seem be the case right now. So this commit ensures that CI is
using this version by naming it explicitly.

[1]: yarnpkg/berry#6398
[2]: nodejs/node#53935
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