Skip to content

Commit c7c3f4c

Browse files
committed
remove find
1 parent a46abe2 commit c7c3f4c

14 files changed

Lines changed: 23 additions & 30 deletions

File tree

src/node/install.sh

Lines changed: 16 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -289,28 +289,23 @@ fi
289289
# Determine the Node.js version using the .nvmrc or .node-version file if present.
290290
if [[ "${NODE_VERSION}" == "project-file" ]]; then
291291
echo "Finding Node version from .nvmrc or .node-version file..."
292-
NODE_VERSION_PATH=$(find . -type f -name ".node-version" | head -n 1)
293-
NVMRC_PATH=$(find . -type f -name ".nvmrc" | head -n 1)
294-
# Used as the default when no file exists or if the file is empty
295-
NODE_VERSION="lts"
296-
if [ -n "$NODE_VERSION_PATH" ]; then
297-
NODE_VERSION_NODE_VERSION_FILE=$(<"$NODE_VERSION_PATH" xargs)
298-
if [ -n "$NODE_VERSION_NODE_VERSION_FILE" ]; then
299-
echo "Using Node version from .node-version file in $NODE_VERSION_PATH: $NODE_VERSION_NODE_VERSION_FILE"
300-
NODE_VERSION="${NODE_VERSION_NODE_VERSION_FILE}"
301-
else
302-
echo "$NODE_VERSION_PATH file is empty. No Node version specified. Using the default: ${NODE_VERSION}."
303-
fi
304-
elif [ -n "$NVMRC_PATH" ]; then
305-
NODE_VERSION_NVMRC=$(<"$NVMRC_PATH" xargs)
306-
if [ -n "$NODE_VERSION_NVMRC" ]; then
307-
echo "Using Node version from .nvmrc file in $NVMRC_PATH: $NODE_VERSION_NVMRC"
308-
NODE_VERSION="${NODE_VERSION_NVMRC}"
309-
else
310-
echo "$NVMRC_PATH file is empty. No Node version specified. Using the default: ${NODE_VERSION}."
292+
NODE_VERSION_FOUND=""
293+
for version_file in ".node-version" ".nvmrc"; do
294+
if [[ -f "$version_file" && -s "$version_file" ]]; then
295+
file_version=$(tr -d '[:space:]' < "$version_file")
296+
if [[ -n "$file_version" ]]; then
297+
echo "Using Node version from $version_file: $file_version"
298+
NODE_VERSION="$file_version"
299+
NODE_VERSION_FOUND="yes"
300+
break
301+
else
302+
echo "$version_file exists but contains only whitespace. Continuing search..."
303+
fi
311304
fi
312-
else
313-
echo "No .node-version or .nvmrc file found. Using the default Node version: ${NODE_VERSION}."
305+
done
306+
if [[ -z "$NODE_VERSION_FOUND" ]]; then
307+
NODE_VERSION="lts"
308+
echo "No Node version found in .nvmrc or .node-version. Using default: lts"
314309
fi
315310
fi
316311

test/node/test_node_project_nodev.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ set -e
55
# Optional: Import test library
66
source dev-container-features-test-lib
77

8-
cd test_node_project_nodev/sample-node
8+
cd test_node_project_nodev
99

1010
# Check that .nvmrc exists
1111
if [ ! -f .node-version ]; then
File renamed without changes.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
FROM debian:11
22

3-
COPY sample-node /tmp/dev-container-features/node_0/sample-node
3+
COPY .node-version /tmp/dev-container-features/node_0/.node-version

test/node/test_node_project_nodev/sample-node/src/hello.js

Lines changed: 0 additions & 1 deletion
This file was deleted.

test/node/test_node_project_nvm_nodev.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ set -e
55
# Optional: Import test library
66
source dev-container-features-test-lib
77

8-
cd test_node_project_nvm_nodev/sample-node
8+
cd test_node_project_nvm_nodev
99

1010
# Check that .node-version exists
1111
if [ ! -f .node-version ]; then

test/node/test_node_project_nvm_nodev/sample-node/.node-version renamed to test/node/test_node_project_nvm_nodev/.node-version

File renamed without changes.
File renamed without changes.
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
FROM debian:11
22

3-
COPY sample-node /tmp/dev-container-features/node_0/sample-node
3+
COPY .nvmrc .node-version /tmp/dev-container-features/node_0/
4+

test/node/test_node_project_nvm_nodev/sample-node/src/hello.js

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)