Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

chore: remove code duplication in custom Node.js version logic (Xenial) #758

Merged
merged 2 commits into from
Mar 24, 2022
+10 −6
Merged

chore: remove code duplication in custom Node.js version logic (Xenial) #758

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
229205e
chore: remove code duplication in custom Node.js version logic
ehmicky Mar 8, 2022
3c4bbf6
Merge branch 'xenial' into chore/refactor-custom-node-xenial
ehmicky Mar 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 10 additions & 6 deletions run-build-functions.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,12 @@ check_python_version() {
fi
}

read_node_version_file() {
local nodeVersionFile="$1"
NODE_VERSION="$(cat "$nodeVersionFile")"
echo "Attempting node version '$NODE_VERSION' from $nodeVersionFile"
}

install_dependencies() {
local defaultNodeVersion=$1
local defaultRubyVersion=$2
Expand Down Expand Up @@ -247,14 +253,12 @@ install_dependencies() {
echo "Finished restoring cached node version"
fi

if [ -f .nvmrc ]
if [ -f ".nvmrc" ]
then
NODE_VERSION=$(cat .nvmrc)
echo "Attempting node version '$NODE_VERSION' from .nvmrc"
elif [ -f .node-version ]
read_node_version_file ".nvmrc"
elif [ -f ".node-version" ]
then
NODE_VERSION=$(cat .node-version)
echo "Attempting node version '$NODE_VERSION' from .node-version"
read_node_version_file ".node-version"
fi

if nvm install --no-progress $NODE_VERSION
Expand Down