-
Notifications
You must be signed in to change notification settings - Fork 120
chore: Update dependencies and benchmark relating code #184
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
Conversation
| @@ -1,23 +1,15 @@ | |||
| // This benchmark project is based on CliFx.Benchmarks. | |||
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.
.editorconfig is recently modified to use utf8-bom.
It need to apply this rule to all cs files by dotnet format command on another PR.
|
I've compared to README's benchmark results that is measured on 2024/5/31. I don't know what cause this performance differences.
It's reproduced by using v5.0.0 tag version of benchmark code. (that using System.CommandLine v5.0.0 benchmark results
|
|
Thank you, checking the latest is also important from a fairness perspective. |
|
Thanks for merging this PR.
If possible, please update |
This PR update package dependencies and update benchmark relating code.
to test System.CommandLine
2.0.0-beta5.Before this PR merged
After this PR merged
Note:
Completed Work Itemscolumns are displayed. BecauseThreadingDiagnoseris added..