Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "preview build" support to Bitrise #5807

Merged
merged 1 commit into from
Mar 8, 2023
Merged

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Feb 21, 2023

Development & PR Process

  1. Follow MetaMask Mobile Coding Standards
  2. Add release-xx label to identify the PR slated for a upcoming release (will be used in release discussion)
  3. Add needs-dev-review label when work is completed
  4. Add needs-qa label when dev review is completed
  5. Add QA Passed label when QA has signed off

Description

The Bitrise workflows for QA builds and e2e tests now support using "preview builds" from the core monorepo. The release builds still won't work with preview builds, which should ensure we don't accidentally include a preview build in a release.

Issue

I don't think there is an issue for this yet, and I'm not sure which work it most directly relates to.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

@Gudahtt

This comment was marked as resolved.

@Gudahtt Gudahtt mentioned this pull request Feb 21, 2023
3 tasks
@Gudahtt Gudahtt force-pushed the consolidate-bitrise-setup branch 5 times, most recently from b7c2d96 to cc2e59f Compare February 28, 2023 20:08
Base automatically changed from consolidate-bitrise-setup to main March 6, 2023 13:08
@Gudahtt Gudahtt force-pushed the bitrise-preview-builds branch 2 times, most recently from 5bcf466 to 7d54bca Compare March 8, 2023 01:12
The Bitrise workflows for QA builds and e2e tests now support using
"preview builds" from the core monorepo [1]. The release builds still
won't work with preview builds, which should ensure we don't
accidentally include a preview build in a release.

[1]: https://github.com/MetaMask/core/blob/main/docs/contributing.md#using-packages-in-other-projects-during-developmenttesting
@Gudahtt Gudahtt marked this pull request as ready for review March 8, 2023 02:00
@Gudahtt Gudahtt requested a review from a team as a code owner March 8, 2023 02:00
@Gudahtt
Copy link
Member Author

Gudahtt commented Mar 8, 2023

This is now ready for review. Test builds on Bitrise are still running, but they've gotten past the install step at least.

Here's a run on this branch: https://app.bitrise.io/build/61f64eaf-9bf1-40bc-bf69-a281d7933eb3
And here's a run on a test branch that is using a preview build from @metamask/core: https://app.bitrise.io/build/ed8fb928-8023-4077-a3b2-b7a80db366c1

Copy link
Member

@gantunesr gantunesr left a comment

Choose a reason for hiding this comment

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

LGTM

@Gudahtt Gudahtt merged commit ccdf2a9 into main Mar 8, 2023
@Gudahtt Gudahtt deleted the bitrise-preview-builds branch March 8, 2023 21:37
@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2023
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.

None yet

2 participants