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

NIT: updated readme for better readability #218

Merged
merged 1 commit into from
Mar 17, 2023
Merged

NIT: updated readme for better readability #218

merged 1 commit into from
Mar 17, 2023

Conversation

anmluo
Copy link
Contributor

@anmluo anmluo commented Mar 17, 2023

Minor changes for readme

Signed-off-by: Andrew Luo <anluo@google.com>
@anmluo anmluo requested a review from a team as a code owner March 17, 2023 21:30
@anmluo anmluo requested review from bradegler and yolocs March 17, 2023 21:30
@sethvargo
Copy link
Contributor

@anmluo - can you enable commit signing on your account? I'll force this merge, but we require all commits to be signed.

@sethvargo sethvargo merged commit a900c55 into main Mar 17, 2023
@sethvargo sethvargo deleted the readme_update branch March 17, 2023 21:46
@anmluo
Copy link
Contributor Author

anmluo commented Mar 17, 2023

@sethvargo I will make all the commits are signed. Already enable it in my account but probably forget to add -S option during commit.

@sethvargo
Copy link
Contributor

Add:

[commit]
  gpgsign = true

[tag]
  gpgsign = true

to your $HOME/.gitconfig :)

@anmluo
Copy link
Contributor Author

anmluo commented Mar 17, 2023

cool, added it, thx for the tip.

sqin2019 pushed a commit that referenced this pull request Apr 6, 2023
Signed-off-by: Andrew Luo <anluo@google.com>
verbanicm pushed a commit that referenced this pull request Jun 14, 2023
Signed-off-by: Andrew Luo <anluo@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants