-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move fast checks into earlier Travis task. #18193
Conversation
Heads up! This PR modifies the following files:
|
@bors-servo r+ Thanks for coming up with this! |
📌 Commit f9dac85 has been approved by |
@bors-servo r- I think |
On 08/22/2017 08:33 PM, Simon Sapin wrote:
@bors-servo <https://github.com/bors-servo> r-
I think |lockfile_changed.sh| only works after running Cargo.
Oh, indeed, great catch :(.
|
The lockfile check has been reverted. |
@bors-servo r+ |
📌 Commit 39f98a0 has been approved by |
Move fast checks into earlier Travis task. This allows quicker feedback about whether the manifest needs updating; otherwise it has to wait until all builds and tests are complete. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18193) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This allows quicker feedback about whether the manifest needs updating; otherwise it has to wait until all builds and tests are complete.
This change is