Skip to content

MK8S-81 - documentation: Add README, LICENSE, CONTRIBUTING and CODEOWNERS documents. - #17

Merged
vdaviot merged 2 commits into
mainfrom
improvement/MK8S-82-documentation
Apr 7, 2026
Merged

MK8S-81 - documentation: Add README, LICENSE, CONTRIBUTING and CODEOWNERS documents.#17
vdaviot merged 2 commits into
mainfrom
improvement/MK8S-82-documentation

Conversation

@vdaviot

@vdaviot vdaviot commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Branch name is not right, should be improvement/MK8S-81-documentation

Signed-off-by: Valentin Daviot <valentin.daviot@scality.com>
@vdaviot vdaviot changed the title MK8S-82 - documentation: Add basic AI generated doc MK8S-81 - documentation: Add basic AI generated doc Apr 1, 2026
Signed-off-by: Valentin Daviot <valentin.daviot@scality.com>
@vdaviot
vdaviot requested a review from a team as a code owner April 2, 2026 13:12
@vdaviot vdaviot changed the title MK8S-81 - documentation: Add basic AI generated doc MK8S-81 - documentation: Add README, LICENSE, CONTRIBUTING and CODEOWNERS documents. Apr 2, 2026
Comment thread CONTRIBUTING.md
4. Run the full validation suite before pushing:

```bash
make manifests generate fmt vet lint test

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.

nit: maybe an umbrella recipe against make that triggers it all would be nice for contributors

@eg-ayoub eg-ayoub left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, I second Alex's comment

@vdaviot

vdaviot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

I don't think we should, we're supposed to stay as close as possible to operator-sdk scaffolding, adding makefile recipe like this would push us in the other direction. @eg-ayoub @ezekiel-alexrod

@vdaviot

vdaviot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

I'm closing this, will address the comment in another PR if need be

@vdaviot vdaviot closed this Apr 7, 2026
@vdaviot vdaviot reopened this Apr 7, 2026
@vdaviot
vdaviot merged commit b4392a0 into main Apr 7, 2026
14 of 15 checks passed
@vdaviot
vdaviot deleted the improvement/MK8S-82-documentation branch April 7, 2026 07:41
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