Skip to content
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

Document JSON message output. #64925

Merged
merged 1 commit into from
Oct 18, 2019
Merged

Document JSON message output. #64925

merged 1 commit into from
Oct 18, 2019

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Sep 30, 2019

This documents the JSON messages in the rustc book.

@rust-highfive
Copy link
Collaborator

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
Contributor

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
Contributor

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
Contributor

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
Collaborator

📣 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