Skip to content

Commit 6d56a70

Browse files
brendankennypaulirish
authored andcommitted
misc: more releasing tweaks (GoogleChrome#9772)
1 parent 75ae0f8 commit 6d56a70

File tree

2 files changed

+4
-11
lines changed

2 files changed

+4
-11
lines changed

docs/releasing.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ and that no new PRs should be merged until you are done.
6060
```sh
6161
# Run the tests.
6262
bash ./lighthouse-core/scripts/release/test.sh
63-
# Change into the pristine folder.
63+
# Change into the newly-created pristine folder.
6464
cd ../lighthouse-pristine
6565
```
6666

@@ -85,7 +85,7 @@ yarn --cwd ~/chromium/src/third_party/blink/renderer/devtools test 'http/tests/d
8585
git diff
8686

8787
# Verify that the Audits panel still works. Consider the new features that have been added.
88-
# If anything is wrong, stop releasing, investigate, and prioritize landing the PR.
88+
# If anything is wrong, stop releasing, investigate, land a fix and start over.
8989

9090
# For bonus points, add some tests covering new features. Either a new test, or an extra
9191
# assertion in an existing test.
@@ -105,7 +105,7 @@ Confirm Lightrider integration will work:
105105

106106
# Verify that Lightrider works properly, and is generating reports fully. Consider the new features that have been added.
107107
# Note: if the changes include proto changes make sure that the API has those new fields.
108-
# If anything is wrong, stop releasing, investigate, and prioritize landing the PR.
108+
# If anything is wrong, stop releasing, investigate, land a fix and start over.
109109

110110
# For bonus points, add some tests covering new features. Either a new test, or an extra
111111
# assertion in an existing test.
@@ -114,10 +114,6 @@ Confirm Lightrider integration will work:
114114
Now that the integrations are confirmed to work, go back to `lighthouse` folder.
115115

116116
```sh
117-
# Run the tests again.
118-
# Note: you can skip this if you didn't need to land any changes.
119-
bash ./lighthouse-core/scripts/release/test.sh
120-
121117
# Prepare the commit, replace x.x.x with the desired version
122118
bash ./lighthouse-core/scripts/release/prepare-commit.sh x.x.x
123119

lighthouse-core/scripts/release/test.sh

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,6 @@ yarn build-all
2323
echo "Running the standard test suite..."
2424
yarn test
2525

26-
echo "Running the smoke tests...."
27-
yarn smoke || yarn smoke || yarn smoke
28-
2926
echo "Testing the CLI..."
3027
yarn start "https://example.com" --view
3128

@@ -39,7 +36,7 @@ cd /tmp/lighthouse-local-test
3936

4037
npm init -y
4138
npm install "$LH_PRISTINE_ROOT/lighthouse-$VERSION.tgz"
42-
npm explore lighthouse -- npm run smoke
39+
npm explore lighthouse -- npm run smoke || npm explore lighthouse -- npm run smoke
4340
npm explore lighthouse -- npm run chrome # try the manual launcher
4441
npm explore lighthouse -- npm run fast -- http://example.com
4542

0 commit comments

Comments
 (0)