Skip to content

Add contributing guidelines to CONTRIBUTING.md#8

Open
jwagantall wants to merge 1 commit intomainfrom
contributing
Open

Add contributing guidelines to CONTRIBUTING.md#8
jwagantall wants to merge 1 commit intomainfrom
contributing

Conversation

@jwagantall
Copy link
Copy Markdown
Contributor

Added a comprehensive contributing guide for the project, including sections on code of conduct, pull requests, sign-off requirements, and how to report bugs or suggest enhancements.

Added a comprehensive contributing guide for the project, including sections on code of conduct, pull requests, sign-off requirements, and how to report bugs or suggest enhancements.

Signed-off-by: Jessica G <jwagantall@linuxfoundation.org>
@jwagantall
Copy link
Copy Markdown
Contributor Author

@matthew1001 , we need this guide too.. however, I used the one we have in hiero as a base and we can modify it to the Paladin project.

This is a little detailed, but feel free to comment on what would you like to change.


It is a recommended best practice to use the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) specification.

8. Enable [GPG signing](https://docs.github.com/en/github/authenticating-to-github/managing-commit-signature-verification/signing-commits) of your commits within the repo.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we are planning to have GPG signing of commits for Paladin. We will require commits to be signed off, but not GPG signed

Copy link
Copy Markdown
Contributor Author

@jwagantall jwagantall Dec 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you ... let me remove that part for now.. however is a good practice to have GPG signatures. Is this something the team is willing to adopt? Has this been discussed before?

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.

2 participants