Skip to content
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

adding hack to add git hashing to report #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jaredeh
Copy link

@jaredeh jaredeh commented May 26, 2023

In the Storage WG meeting I made a suggestion for having the benchmark check the code. This is an example of how that could work. It will add a 'git' section to the .json report with a git hash and if relevant a diff showing what has been changed.

@jaredeh jaredeh requested a review from a team as a code owner May 26, 2023 22:45
@github-actions
Copy link

MLCommons CLA bot:
Thank you very much for your submission, we really appreciate it. Before we can accept your contribution, we ask that you sign the MLCommons CLA (Apache 2). Please use this [Google form] (https://forms.gle/Ew1KkBVpyeJDuRw67) to initiate authorization. If you are from an MLCommons member organization, we will request that you be added to the CLA. If you are not from a member organization, we will email you a CLA to sign. For any questions, please contact [email protected].
0 out of 1 committers have signed the MLCommons CLA.
@jaredeh
You can retrigger this bot by commenting recheck in this Pull Request

@jaredeh
Copy link
Author

jaredeh commented May 26, 2023

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant