-
Notifications
You must be signed in to change notification settings - Fork 323
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
Experiment: Reporting test failures as annotations #10821
base: develop
Are you sure you want to change the base?
Conversation
…ns (and also make the output more readable...)
@@ -223,7 +223,9 @@ impl BuiltEnso { | |||
if errors.is_empty() { | |||
Ok(()) | |||
} else { | |||
error!("{} test suit(s) failed.", errors.len()); | |||
let summary = errors.as_slice().iter().map(|e| e.to_string()).collect::<Vec<_>>().join(", "); | |||
println!("::error title=Failed Standard Library Tests::{} test suite(s) failed: {}", errors.len(), summary); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see a line starting with ::
- that must be a GitHub Actions command line! Where is the syntax documented?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am glad we ended up doing the same - e.g. using println!
to print the command line without a prefix. Just like at my PR.
I did not manage to cleanup and finish this yet because I was focusing on more pressing tasks. I will re-visit this once I'm back from vacation. |
Pull Request Description
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.