-
Notifications
You must be signed in to change notification settings - Fork 1.5k
docs: Add some more detail to the AI guidelines #7945
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
docs: Add some more detail to the AI guidelines #7945
Conversation
Some ideas from the team, some from myself and the rest from these resources here: * https://github.com/open-telemetry/community/blob/main/policies/genai.md * https://www.linuxfoundation.org/legal/generative-ai Signed-off-by: Charlie Egan <[email protected]>
✅ Deploy Preview for openpolicyagent ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
johanfylling
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
| - **Commits:** All code must be yours to contribute and commits must be signed off (see | ||
| [Commit Messages](#commit-messages) below). | ||
| Related commits must be squashed before they are merged (this can be done in | ||
| the PR UI on GitHub). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be done in the PR UI on GitHub
This is an option for the one merging the PR, but is it an option for the contributor whom these docs are for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was just here before, but I wondered the same... I saw no harm in leaving it here as something for them to be aware of as they work that eventually their commits will be squashed?
| - Correct maintainers when you think they've misunderstood something. | ||
|
|
||
| Together we can learn and build a better OPA! | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
Thanks for the review Johan! |
Some ideas from the team, some from myself and the rest from these resources here: