-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: add JSON compatibility for forge test --summary +/ --detailed
+ apply consistent table styling
#9485
Conversation
…strictly Markdown compatible
--summary
and --detailed
+ apply consistent table stylingforge test --summary +/ --detailed
+ apply consistent table styling
…erent branching and could be derived from JSON
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.
lgtm, consistent summaries looks great!
* fix(generate): update for chain addition * fix: use nativeChain to compare * fix(ci): use `forge inspect --json` See foundry-rs/foundry#9485
Hey all, previously the table formatting was markdown, and we could pipe it directly into |
Hi @adamegyed We recently added broad support for JSON serialization of the output rendered to the console using the global |
Motivation
Improves JSON compatibility for
forge test
, improve consistency of table renderingSolution
--summary
and--summary
+--detailed
SummaryReporter
to beCoverageSummaryReporter