Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

ci: update release & publish actions #200

Merged
merged 3 commits into from
Apr 22, 2024

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Feb 23, 2024

  • ci: update MetaMask/action-npm-publish from v2 to v4
    • Requires setting SLACK_WEBHOOK_URL repo env secret
  • ci: update MetaMask/action-publish-release-action from v2 to v3
  • ci: update MetaMask/action-create-release-pr from v1 to v3

Ported from https://github.com/MetaMask/metamask-module-template/blob/main/.github/workflows/

@legobeat legobeat marked this pull request as ready for review February 23, 2024 03:17
@legobeat legobeat requested a review from a team as a code owner February 23, 2024 03:17
danroc
danroc previously approved these changes Mar 21, 2024
@mcmire
Copy link

mcmire commented Mar 21, 2024

I checked the secrets and it looks like SLACK_WEBHOOK_URL is already set on the GitHub org level, so we shouldn't need to do anything there.

mcmire
mcmire previously approved these changes Mar 21, 2024
@danroc
Copy link
Contributor

danroc commented Mar 22, 2024

The CI seems to be stuck and I cannot merge this PR. @legobeat can you push an empty commit to trigger it again?

@legobeat legobeat force-pushed the ci-publish-update branch 3 times, most recently from 623e43c to 39bc77d Compare March 27, 2024 23:31
- Requires setting `SLACK_WEBHOOK_URL` repo env secret
@legobeat
Copy link
Contributor Author

The CI seems to be stuck and I cannot merge this PR. @legobeat can you push an empty commit to trigger it again?

Passing now!

@legobeat legobeat requested review from mcmire, Akaryatrh and a team April 15, 2024 04:34
@danroc danroc merged commit 8dcdfad into MetaMask:main Apr 22, 2024
19 checks passed
@legobeat legobeat deleted the ci-publish-update branch April 22, 2024 08:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants