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

Implementing "<test_binary> --list --format json" for use by IDE test explorers / runners #592

Closed
parthopdas opened this issue Feb 19, 2023 · 4 comments
Labels
major-change A proposal to make a major change to rustc major-change-accepted A major change proposal that was accepted T-compiler Add this label so rfcbot knows to poll the compiler team

Comments

@parthopdas
Copy link

Proposal

pre-RFC: 18308
Feature Name: Implementing "<test_binary> --list --format json"
Start Date: 2023-02-08
RFC PR: none yet (based however on discussions in #107307 2)
Rust Issue: rust-lang/rust#107307 3

<test_binary> --format pretty|terse|json|junit already exists. (See #49359)

As does <test_binary> --list.

This proposal put both together i.e. <test_binary> --list --format json for use by IDE test explorers / runners for superior user experience.

Until stabilization, a topic for an upcoming RFC, this will be available under -Zunstable-options.

Mentors or Reviewers

@bjorn3 @oli-obk @Manishearth

@parthopdas parthopdas added major-change A proposal to make a major change to rustc T-compiler Add this label so rfcbot knows to poll the compiler team labels Feb 19, 2023
@rustbot
Copy link
Collaborator

rustbot commented Feb 19, 2023

This issue is not meant to be used for technical discussion. There is a Zulip stream for that. Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.

cc @rust-lang/compiler @rust-lang/compiler-contributors

@rustbot rustbot added the to-announce Announce this issue on triage meeting label Feb 19, 2023
@oli-obk
Copy link
Contributor

oli-obk commented Feb 20, 2023

@rustbot second

@rustbot rustbot added the final-comment-period The FCP has started, most (if not all) team members are in agreement label Feb 20, 2023
@apiraino apiraino removed the to-announce Announce this issue on triage meeting label Feb 24, 2023
@parthopdas
Copy link
Author

@oli-obk i believe the 10 day comment period has been completed. if there weren't any concerns raised, please can you approve the PR rust-lang/rust#108148?

@apiraino
Copy link
Contributor

apiraino commented Mar 7, 2023

@rustbot label -final-comment-period +major-change-accepted

@apiraino apiraino closed this as completed Mar 7, 2023
@rustbot rustbot added major-change-accepted A major change proposal that was accepted to-announce Announce this issue on triage meeting and removed final-comment-period The FCP has started, most (if not all) team members are in agreement labels Mar 7, 2023
@apiraino apiraino removed the to-announce Announce this issue on triage meeting label Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major-change A proposal to make a major change to rustc major-change-accepted A major change proposal that was accepted T-compiler Add this label so rfcbot knows to poll the compiler team
Projects
None yet
Development

No branches or pull requests

4 participants