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

Update README.md #1216

Closed
wants to merge 1 commit into from
Closed

Update README.md #1216

wants to merge 1 commit into from

Conversation

baibhavKumar1
Copy link

debugged broken link

@baibhavKumar1 baibhavKumar1 requested a review from a team as a code owner July 11, 2023 10:53
@mtardy mtardy requested review from mtardy and removed request for olsajiri July 11, 2023 10:55
@mtardy
Copy link
Member

mtardy commented Jul 11, 2023

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 git commit -s to write your commit message. You can use git commit --amend -s to add the signature to existing commits.

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 --global flag after config to configure this by default for repositories on your host.

Then you can use for new commits:

git commit -s

Or for existing commits on which you want to add the signed-off-by statement:

git commit --amend -s

For more information, see this extract from git-commit(1) man page regarding the -s flag:

-s, --signoff, --no-signoff
   Add a Signed-off-by trailer by the committer at the end of the commit log
   message. The meaning of a signoff depends on the project to which you're
   committing. For example, it may certify that the committer has the rights to
   submit the work under the project's license or agrees to some contributor
   representation, such as a Developer Certificate of Origin. (See
   http://developercertificate.org for the one used by the Linux kernel and Git
   projects.) Consult the documentation or leadership of the project to which
   you're contributing to understand how the signoffs are used in that project.

   The --no-signoff option can be used to countermand an earlier --signoff
   option on the command line.

Can you also write your commit message with something like README: fix tracing policy doc broken link

@mtardy
Copy link
Member

mtardy commented Jul 11, 2023

Can you squash your 3 commits into one?

@baibhavKumar1
Copy link
Author

Hi, sorry for the mess, can you please help me with that.

@mtardy
Copy link
Member

mtardy commented Jul 11, 2023

Still we need a real name in the author and signature field.

@baibhavKumar1
Copy link
Author

What needs to be done for that? My name is "Baibhav" , but I don't have github account with that name.

@mtardy
Copy link
Member

mtardy commented Jul 11, 2023

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 git commit -s to write your commit message. You can use git commit --amend -s to add the signature to existing commits.

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 --global flag after config to configure this by default for repositories on your host.

Then you can use for new commits:

git commit -s

Or for existing commits on which you want to add the signed-off-by statement:

git commit --amend -s

For more information, see this extract from git-commit(1) man page regarding the -s flag:

-s, --signoff, --no-signoff
   Add a Signed-off-by trailer by the committer at the end of the commit log
   message. The meaning of a signoff depends on the project to which you're
   committing. For example, it may certify that the committer has the rights to
   submit the work under the project's license or agrees to some contributor
   representation, such as a Developer Certificate of Origin. (See
   http://developercertificate.org for the one used by the Linux kernel and Git
   projects.) Consult the documentation or leadership of the project to which
   you're contributing to understand how the signoffs are used in that project.

   The --no-signoff option can be used to countermand an earlier --signoff
   option on the command line.

@baibhavKumar1
Copy link
Author

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?

@jrfastab
Copy link
Contributor

@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 git commit --ammend -s and push again and should be good to go.

@mtardy
Copy link
Member

mtardy commented Jul 24, 2023

Fixed by #1270.

@mtardy mtardy closed this Jul 24, 2023
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.

3 participants