-
-
Notifications
You must be signed in to change notification settings - Fork 1k
passed args to benchmark runner #1292
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
did not wrote test cases yet, need a review from @adamsitnik, |
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.
Thank you @chan18 ! The PR looks very promising, please take a look at my comments and address them.
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.
Looks good to me!
Could you please add the public static Summary[] Run(Assembly assembly, IConfig config = null,string[] args = null)
overload and fix the formatting issues?
Co-Authored-By: Adam Sitnik <[email protected]>
Co-Authored-By: Adam Sitnik <[email protected]>
Co-Authored-By: Adam Sitnik <[email protected]>
Co-Authored-By: Adam Sitnik <[email protected]>
BenchmarkDotNet/src/BenchmarkDotNet/Running/BenchmarkRunnerDirty.cs Lines 82 to 118 in 5589986
needs another overload. I am thinking to create another function to move the code that parses args and use it in two different functions, Something like this in below.
|
… types, IEnumerable<Assembly> assemblies, IConfig config, string[] args)
hi, @adamsitnik please review it. Let me know any other changes or improvements you want to make! |
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.
Co-Authored-By: Adam Sitnik <[email protected]>
marge from original repo to my fork.
The build fails with following compilation error:
Could you please fix it? |
Fixed it, Can you please check. let me know if you have anyother change that i need to make. |
fixed compiled error.
# Conflicts: # src/BenchmarkDotNet/Running/BenchmarkSwitcher.cs
fixes #1290