Skip to content

chore: set nvmrc to 18.18.2#8960

Merged
jpcloureiro merged 1 commit into
mainfrom
fix/bitrise-asdf-nodejs
Mar 15, 2024
Merged

chore: set nvmrc to 18.18.2#8960
jpcloureiro merged 1 commit into
mainfrom
fix/bitrise-asdf-nodejs

Conversation

@jpcloureiro

Copy link
Copy Markdown
Contributor

Description

set .nvmrc node version in sync with NODE_VERSION on bitrise.yml

This fixes bitrise builds on mac os machines

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

  • I’ve followed MetaMask Coding Standards.
  • I've clearly explained what problem this PR is solving and how it is solved.
  • I've linked related issues
  • I've included manual testing steps
  • I've included screenshots/recordings if applicable
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
  • I’ve properly set the pull request status:
    • In case it's not yet "ready for review", I've set it to "draft".
    • In case it's "ready for review", I've changed it from "draft" to "non-draft".

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@jpcloureiro jpcloureiro requested a review from a team as a code owner March 15, 2024 16:31
@github-actions

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@NicolasMassart NicolasMassart left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

@tommasini tommasini added the team-mobile-platform Mobile Platform team label Mar 15, 2024
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@jpcloureiro jpcloureiro merged commit 1630a58 into main Mar 15, 2024
@jpcloureiro jpcloureiro deleted the fix/bitrise-asdf-nodejs branch March 15, 2024 16:44
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 15, 2024
@metamaskbot metamaskbot added the release-7.19.0 Issue or pull request that will be included in release 7.19.0 label Mar 15, 2024
@leotm

leotm commented Mar 16, 2024

Copy link
Copy Markdown
Contributor

doh should have considered bitrise earlier 😅 thank you chaps

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.19.0 Issue or pull request that will be included in release 7.19.0 team-mobile-platform Mobile Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants