Skip to content

toolstate: Fixed detection of changed submodule, and other fixes. - #52004

Merged
bors merged 3 commits into
rust-lang:masterfrom
kennytm:toolstate-fixes
Jul 3, 2018
Merged

toolstate: Fixed detection of changed submodule, and other fixes.#52004
bors merged 3 commits into
rust-lang:masterfrom
kennytm:toolstate-fixes

Conversation

@kennytm

@kennytmkennytm commented Jul 2, 2018

Copy link
Copy Markdown
Member
  1. Make sure that if a submodule is updated but failed to test-pass, we'll block the merge.
  2. Make sure failure on external docs (nomicon/RBE/etc) are properly checked.
  3. If the commit message starts with "Update RLS" (or clippy etc), automatically run the "tools" job on the PR, so that we could know if the update failed before merging.

@rust-highfive

This comment has been minimized.

@rust-highfive

This comment has been minimized.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 2, 2018
@rust-highfive

This comment has been minimized.

@kennytm
kennytmforce-pushed the toolstate-fixes branch 2 times, most recently from 8b01e58 to 7274e50CompareJuly 2, 2018 21:15
kennytm added 2 commits July 3, 2018 05:22
This ensure we do block the tools when they are broken during an update.
@kennytmkennytm changed the title [WIP] toolstate: Fix detection of changed submodule.toolstate: Fixed detection of changed submodule, and other fixes.Jul 2, 2018
@kennytm

Copy link
Copy Markdown
MemberAuthor

r? @Mark-Simulacrum

@Mark-Simulacrum

Copy link
Copy Markdown
Member

Looks good to me.

@bors r+

@bors

bors commented Jul 2, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 689cffa has been approved by Mark-Simulacrum

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 2, 2018
emilyalbini added a commit to emilyalbini/rust that referenced this pull request Jul 3, 2018
…ulacrum
toolstate: Fixed detection of changed submodule, and other fixes.
1. Make sure that if a submodule is updated but failed to test-pass, we'll block the merge.
2. Make sure failure on external docs (nomicon/RBE/etc) are properly checked.
3. If the commit message starts with "Update RLS" (or clippy etc), automatically run the "tools" job on the PR, so that we could know if the update failed before merging.
emilyalbini added a commit to emilyalbini/rust that referenced this pull request Jul 3, 2018
…ulacrum
toolstate: Fixed detection of changed submodule, and other fixes.
1. Make sure that if a submodule is updated but failed to test-pass, we'll block the merge.
2. Make sure failure on external docs (nomicon/RBE/etc) are properly checked.
3. If the commit message starts with "Update RLS" (or clippy etc), automatically run the "tools" job on the PR, so that we could know if the update failed before merging.
bors added a commit that referenced this pull request Jul 3, 2018
Rollup of 13 pull requests
Successful merges:
- #51548 (Initialize LLVM's AMDGPU target machine, if available.)
- #51809 (Add read_exact_at and write_all_at methods to FileExt on unix)
- #51914 (add outlives annotations to `BTreeMap`)
- #51958 (Show known meta items in unknown meta items error)
- #51973 (Make Stdio handle UnwindSafe)
- #51977 (bootstrap: tests should use rustc from config.toml)
- #51978 (Do not suggest changes to str literal if it isn't one)
- #51979 (Get rid of `TyImplTraitExistential`)
- #51980 (Emit column info in debuginfo for non msvc like targets)
- #51982 (incr.comp.: Take names of children into account when computing the ICH of a module's HIR.)
- #51997 (add entry for cargo-metadata feature to RELEASES)
- #52004 (toolstate: Fixed detection of changed submodule, and other fixes.)
- #52006 ( Change --keep-stage to apply more often)
Failed merges:
r? @ghost
@bors
bors merged commit 689cffa into rust-lang:masterJul 3, 2018
@kennytm
kennytm deleted the toolstate-fixes branch July 3, 2018 14:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@kennytm@rust-highfive@Mark-Simulacrum@bors@nikomatsakis