Skip to content

feat: added CONTRIBUTING.md for OKDP contribution workflow#4

Open
jpmewenemesse wants to merge 5 commits intoOKDP:mainfrom
jpmewenemesse:feat/added-contributing-file
Open

feat: added CONTRIBUTING.md for OKDP contribution workflow#4
jpmewenemesse wants to merge 5 commits intoOKDP:mainfrom
jpmewenemesse:feat/added-contributing-file

Conversation

@jpmewenemesse
Copy link
Copy Markdown
Collaborator

Purpose

This PR covers the contribution workflow for OKDP repositories:

  1. issue reporting
  2. fork-based vs direct branch workflow depending on repo visibility
  3. commit message conventions
  4. PR review process

@jpmewenemesse jpmewenemesse self-assigned this May 4, 2026
@jpmewenemesse jpmewenemesse linked an issue May 4, 2026 that may be closed by this pull request
13 tasks
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md
jpmewenemesse and others added 3 commits May 5, 2026 15:56
Co-authored-by: Romain Pignolet <romain.pignolet.pro@gmail.com>
Added OKDP to make sure we are talking about OKDP contributors meeting

Co-authored-by: Romain Pignolet <romain.pignolet.pro@gmail.com>
…ic as possible, added git commit --fixup tip
Comment thread CONTRIBUTING.md
| Repository | What to Contribute |
| ----------------------------------------------------------- | ---------------------------------------------- |
| [OKDP/OKDP](https://github.com/OKDP/OKDP) | Project-level docs, governance, roadmap |
| [OKDP/helm-handbook](https://github.com/OKDP/helm-handbook) | Helm chart install guides and values |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

pas encore public, donc a supprimer

Comment thread CONTRIBUTING.md
1. **Fork** the repository on GitHub
2. Clone your fork and create a feature branch from `main`
3. Make your changes
4. Ensure CI passes, if configured (lint, tests, build)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

non-explicit action described here. Could be misleading

@Pierrotws
Copy link
Copy Markdown

What about AI usage, is it allowed ?
If not : what are consequences of hidden usage ? should set a checkbox to confirm at PR that AI has not been used.
Else : should the AI set as co-author ?
Example, in end of commit message

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@jpmewenemesse
Copy link
Copy Markdown
Collaborator Author

@Pierrotws I think your question is worth adressing. But for me,

  • Currently public sector does not officially approve AI usage like Claude or Gemini.... at least at DGFiP, this is not allowed.

More broadly, I dont think enforcement at contribution level is necessary or appropriate. AI assisted is another productivity tool like VSCode or Linter and contributors remain fully responsible for what they submit.

  • Am not sure the CLA currently in progress integrate an AI as co-author or responsability is fully human (cc @lioneloh ).

Where I think there is more interesting conversation is at the project level. Maybe OKDP can think of integrating AI capabilities (AGENTS, Skills, automated review assistance...) like FluxCD AGENTS.md or agents skills rather than governing how individual contributors use their own tools? It is worth discussing at next TOSIT meeting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[governance] Create OKDP/.github org-level community health files

4 participants