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

Add release process for common package#312

Merged
cryptotavares merged 5 commits intomain-desktopfrom
chore/244-add-release-process-for-common-package
Dec 16, 2022
Merged

Add release process for common package#312
cryptotavares merged 5 commits intomain-desktopfrom
chore/244-add-release-process-for-common-package

Conversation

@cryptotavares
Copy link
Copy Markdown
Contributor

@cryptotavares cryptotavares commented Dec 12, 2022

Context

Added a release workflow for the common package (@MetaMask/desktop).

Common package

  • added @MetaMask/auto-changelog and patched it to consider common/[version] git tags.
  • Initialised the changelog file

CI

  • Common package - added github action to create a release (github repo) and publish to npm, when there's a push to main-desktop on the packages/common/ directory and the latest commit has the following pattern common/[version].
    Ended up not using any of MetaMask actions as none of them support independent monorepo package releases.

@cryptotavares cryptotavares linked an issue Dec 12, 2022 that may be closed by this pull request
@socket-security
Copy link
Copy Markdown

socket-security bot commented Dec 12, 2022

Socket Security Pull Request Report

👍 No new dependency issues detected in pull request

Pull request report summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script confusion ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues
Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@2.4.2

Powered by socket.dev

@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch from 66bd406 to bf8d22d Compare December 12, 2022 23:12
@cryptotavares cryptotavares marked this pull request as ready for review December 12, 2022 23:17
@cryptotavares cryptotavares requested a review from a team December 12, 2022 23:17
@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch from bf8d22d to 57cb337 Compare December 12, 2022 23:30
@cryptotavares cryptotavares changed the title Chore/244 add release process for common package Add release process for common package Dec 13, 2022
@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch 3 times, most recently from a43ce3a to b131b41 Compare December 14, 2022 12:50
@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch 4 times, most recently from 907ba35 to 86a297b Compare December 15, 2022 14:50
@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch 2 times, most recently from fafddcf to 268215b Compare December 16, 2022 11:22
Initialise changelog to common package.
The changelog generation should follow
MetaMask repos standard.
We are not using MetaMask template yet,
because it does not support monorepos with
packages with independent versioning
Follow the standard to create
a release branch within MetaMask
@cryptotavares cryptotavares force-pushed the chore/244-add-release-process-for-common-package branch from 268215b to c88cb14 Compare December 16, 2022 11:41
@cryptotavares cryptotavares merged commit edee070 into main-desktop Dec 16, 2022
@cryptotavares cryptotavares deleted the chore/244-add-release-process-for-common-package branch December 16, 2022 12:49
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.

Add release process for @metamask/desktop package

4 participants