Skip to content

Commit 55b03eb

Browse files
committed
Minor tweaks to ci-rush.
1 parent d291568 commit 55b03eb

File tree

3 files changed

+13
-11
lines changed

3 files changed

+13
-11
lines changed

.travis.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,14 @@ script:
77
- set -e
88
- echo 'Checking change file...' && echo -en 'travis_fold:start:check\\r'
99
- git fetch origin master:refs/remotes/origin/master -a
10-
- node common/temp/local-rush/node_modules/@microsoft/rush/lib/start.js change -v
10+
- node common/temp/ci-rush/node_modules/@microsoft/rush/lib/start.js change -v
1111
- echo -en 'travis_fold:end:check\\r'
1212
- echo 'Installing...' && echo -en 'travis_fold:start:install\\r'
13-
- node common/temp/local-rush/node_modules/@microsoft/rush/lib/start.js install --bypass-policy
13+
- node common/temp/ci-rush/node_modules/@microsoft/rush/lib/start.js install --bypass-policy
1414
- echo -en 'travis_fold:end:install\\r'
1515
- echo 'Running rush check' && echo -en 'travis_fold:start:check\\r'
16-
- node common/temp/local-rush/node_modules/@microsoft/rush/lib/start.js check
16+
- node common/temp/ci-rush/node_modules/@microsoft/rush/lib/start.js check
1717
- echo -en 'travis_fold:end:check\\r'
1818
- echo 'Building...' && echo -en 'travis_fold:start:build\\r'
19-
- node common/temp/local-rush/node_modules/@microsoft/rush/lib/start.js rebuild --verbose --production
19+
- node common/temp/ci-rush/node_modules/@microsoft/rush/lib/start.js rebuild --verbose --production
2020
- echo -en 'travis_fold:end:build\\r'

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ if (!fs.existsSync(npmPath)) {
7575
process.exit(1);
7676
}
7777

78-
const rushPathParts: string[] = ['common', 'temp', 'local-rush'];
78+
const rushPathParts: string[] = ['common', 'temp', 'ci-rush'];
7979
let rushPath: string = rushJsonDirectory;
8080
for (const rushPathPart of rushPathParts) {
8181
rushPath = path.join(rushPath, rushPathPart);
@@ -126,7 +126,8 @@ if (!installedVersionValid || installedVersion !== expectedVersion) {
126126
const rushNpmrcPath: string = path.join(rushPath, '.npmrc');
127127
if (fs.existsSync(npmrcPath)) {
128128
try {
129-
const npmrcFileLines: string[] = fs.readFileSync(npmrcPath).toString().split('\n').map((line) => line.trim());
129+
let npmrcFileLines: string[] = fs.readFileSync(npmrcPath).toString().split('\n');
130+
npmrcFileLines = npmrcFileLines.map((line) => (line || '').trim());
130131
const resultLines: string[] = [];
131132
// Trim out lines that reference environment variables that aren't defined
132133
for (const line of npmrcFileLines) {
@@ -154,7 +155,7 @@ if (!installedVersionValid || installedVersion !== expectedVersion) {
154155
}
155156

156157
const packageContents: IPackageJson = {
157-
'name': 'local-rush',
158+
'name': 'ci-rush',
158159
'version': '0.0.0',
159160
'dependencies': {
160161
[PACKAGE_NAME]: expectedVersion

common/scripts/ci-rush.js

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

0 commit comments

Comments
 (0)