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

[TeamCity] - Set node default version on the prepare job #45722

Merged
merged 1 commit into from
Sep 17, 2020

Conversation

scinos
Copy link
Contributor

@scinos scinos commented Sep 17, 2020

Changes

Update the "Prepare environment" to install node from .nvmr. Turns out nvm.sh --install installs the version configured as 'default' in nvm, not the version in .nvmrc.

The process is now:

  • In prepare job:

    • Load nvm.sh, do not try to use .nvmrc (in case it points to a version not installed yet)
    • nvm install will install and use the version indicated in .nvmrc
    • Set the version in .nvmrc as the default version for nvm.
  • In other jobs (unchanged):

    • Load nvm.sh. Will try use the version from .nvmrc, which is already installed.

Testing instructions

Validate that TeamCity build passes.

@matticbot
Copy link
Contributor

@scinos scinos requested a review from a team September 17, 2020 10:27
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Sep 17, 2020
@scinos scinos self-assigned this Sep 17, 2020
@scinos scinos added Build TeamCity Changes in TeamCity configuration Expedited Review labels Sep 17, 2020
@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

Copy link
Member

@tyxla tyxla left a comment

Choose a reason for hiding this comment

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

🚢

@scinos scinos merged commit ff29422 into master Sep 17, 2020
@scinos scinos deleted the fix/teamcity-install-node branch September 17, 2020 11:54
@matticbot matticbot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Sep 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Expedited Review TeamCity Changes in TeamCity configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants