-
Notifications
You must be signed in to change notification settings - Fork 41
Add a guide to creating a separate module for benchmarks #175
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
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.
And the title should be Add a guide for creating
instead of to
?
3. **Register Benchmark Source Set** | ||
|
||
Register your benchmark source set. This informs the kotlinx-benchmark tool | ||
that benchmarks reside within this source set and need to be executed accordingly. |
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.
that benchmarks reside within this source set and need to be executed accordingly. | |
about benchmarks reside within this source set and need to be executed accordingly. |
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 not sure if the change is grammatically correct.
I surfed the internet, and it seems both prepositions are fine to use in this case. |
5ef9a3f
to
0e9ce88
Compare
… docs/separate-benchmark-module.md Co-authored-by: Filipp Zhinkin <[email protected]> Co-authored-by: Zongle Wang <[email protected]>
90c20ae
to
d615cf3
Compare
Followup to #170
The changes were developed within GSoC 2023 program by our mentee @wldeh.