-
-
Notifications
You must be signed in to change notification settings - Fork 508
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
feat(cli): add github
and junit
reporters
#3029
Conversation
CodSpeed Performance ReportMerging #3029 will not alter performanceComparing Summary
|
@@ -43,6 +43,7 @@ dashmap = { workspace = true } | |||
hdrhistogram = { version = "7.5.4", default-features = false } | |||
indexmap = { workspace = true } | |||
lazy_static = { workspace = true } | |||
quick-junit = "0.4.0" |
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.
Should this be on the root cargo.toml, and have workspace = true
? Although, I'm ok with here as well, since I assume this crate will logically always belong to biome_cli
.
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.
Yeah, it only belongs to biome_cli
so there's so need to make it available to the entire workspace
d71a8c9
to
dd4ca40
Compare
Summary
This PR adds
github
andjunit
reporters.The
junit
reporter is heavily inspired by how Ruff does it.Test Plan
I added new test cases