Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Sep 30, 2019

This documents the JSON messages in the rustc book.

@rust-highfive
Copy link
Contributor

r? @QuietMisdreavus

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 30, 2019
@rust-highfive

This comment has been minimized.

@JohnCSimon
Copy link
Member

Ping from triage
@QuietMisdreavus Can you please review this?

@Mark-Simulacrum
Copy link
Member

@bors r+

I believe that this documentation is not adding to our stability guarantees as it is explicit about what we guarantee (new fields, new variants can be added). In any case, I consider rustc book to be somewhat non-normative, though maybe this is not true in practice. I think the JSON interface of rustc we are somewhat less likely to care about breaking (since it affects build tools, rather than source code, and less build tools exist).

@bors
Copy link
Collaborator

bors commented Oct 17, 2019

📌 Commit f04ea6c has been approved by Mark-Simulacrum

@bors bors 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 Oct 17, 2019
Centril added a commit to Centril/rust that referenced this pull request Oct 18, 2019
Document JSON message output.

This documents the JSON messages in the rustc book.
tmandry added a commit to tmandry/rust that referenced this pull request Oct 18, 2019
Document JSON message output.

This documents the JSON messages in the rustc book.
tmandry added a commit to tmandry/rust that referenced this pull request Oct 18, 2019
Document JSON message output.

This documents the JSON messages in the rustc book.
@bors
Copy link
Collaborator

bors commented Oct 18, 2019

⌛ Testing commit f04ea6c with merge 518deda...

bors added a commit that referenced this pull request Oct 18, 2019
Document JSON message output.

This documents the JSON messages in the rustc book.
@bors
Copy link
Collaborator

bors commented Oct 18, 2019

☀️ Test successful - checks-azure
Approved by: Mark-Simulacrum
Pushing 518deda to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Oct 18, 2019
@bors bors merged commit f04ea6c into rust-lang:master Oct 18, 2019
@rust-highfive
Copy link
Contributor

📣 Toolstate changed by #64925!

Tested on commit 518deda.
Direct link to PR: #64925

💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Oct 18, 2019
Tested on commit rust-lang/rust@518deda.
Direct link to PR: <rust-lang/rust#64925>

💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: 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.

6 participants