10

Add RFC for providing ignore test message by yanganto · Pull Request #3217 · rus...

 3 years ago
source link: https://github.com/rust-lang/rfcs/pull/3217
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

Conversation

Copy link

yanganto commented 10 days ago

edited

This RFC proposes to add ignore message into the summary as a result of cargo test.

running 2 tests
test tests::test_ignored ... ignored, not yet implemented
test tests::test_works ... ok
test result: ok. 1 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out; finished in 0.00s

Rendered

Previous discussion on IRLO

Copy link

Contributor

ehuss commented 9 days ago

FWIW, I don't feel like this really needs an RFC. This is just a presentation issue, which can be changed at any time. I think just opening a PR to change it would be sufficient. However, I'm not on the libs team, so they may feel differently. You can try asking on Zulip (#t-libs).

Copy link

Contributor

nikomatsakis commented 7 days ago

I agree with @ehuss that this doesn't require an RFC, but I also think this would be a great addition!

Copy link

Author

yanganto commented 7 days ago

Thanks, I try to work on the PR.
There are some test cases that fail and I still have no idea how to fix them.
I am really appreciated that if someone is willing to help with this.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK