X Tutup
Skip to content

Commit 4e309c0

Browse files
committed
Minor fix to ci-rush to correctly detect environment variables.
1 parent 4514bd4 commit 4e309c0

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

apps/rush-lib/src/scripts/ci-rush.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,9 @@ if (!installedVersionValid || installedVersion !== expectedVersion) {
134134
const environmentVariables: string[] | null = line.match(/\$\{([^\}]+)\}/g);
135135
let lineShouldBeTrimmed: boolean = false;
136136
if (environmentVariables) {
137-
for (const environmentVariable of environmentVariables) {
138-
if (!process.env[environmentVariable]) {
137+
for (const token of environmentVariables) {
138+
const environmentVariableName: string = token.substring(2, token.length - 1);
139+
if (!process.env[environmentVariableName]) {
139140
lineShouldBeTrimmed = true;
140141
break;
141142
}

common/scripts/ci-rush.js

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)
X Tutup