-
Notifications
You must be signed in to change notification settings - Fork 360
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
Update README.md #1216
Update README.md #1216
Conversation
Thanks for the patch! However, it looks like your commit is not signed-off, to improve tracking of who did what, the Tetragon project uses a "sign-off" procedure which you can learn more about in the Cilium project documentation. Concretely it consists of adding your real name and an email address to your git config and using See more details on how to comply with the "sign-off" procedure
To add your name and email to your git config: git config user.email "[email protected]"
git config user.name "firstname lastname" Note: add the Then you can use for new commits: git commit -s Or for existing commits on which you want to add the git commit --amend -s For more information, see this extract from
Can you also write your commit message with something like |
Can you squash your 3 commits into one? |
Hi, sorry for the mess, can you please help me with that. |
Signed-off-by: Wholesomebruh <[email protected]>
Still we need a real name in the author and signature field. |
What needs to be done for that? My name is "Baibhav" , but I don't have github account with that name. |
Please take a look at this again: Thanks for the patch! However, it looks like your commit is not signed-off, to improve tracking of who did what, the Tetragon project uses a "sign-off" procedure which you can learn more about in the Cilium project documentation. Concretely it consists of adding your real name and an email address to your git config and using See more details on how to comply with the "sign-off" procedure
To add your name and email to your git config: git config user.email "[email protected]"
git config user.name "firstname lastname" Note: add the Then you can use for new commits: git commit -s Or for existing commits on which you want to add the git commit --amend -s For more information, see this extract from
|
I did the signature and commit message in last commit but then I did git push, is it required? Or just git commit --amend --no-edit -s will do? |
@Wholesomebruh hi! patch looks good just need a real name on the signed off by. I think to do this open your .gitconfig and change the name to a real name. And then do the |
Fixed by #1270. |
debugged broken link