Skip to content

docs(ISV-7458): add governance - #21

Merged
BorekZnovustvoritel merged 1 commit into
mainfrom
ISV-7458
Sep 9, 2026
Merged

BorekZnovustvoritel merged 1 commit into
mainfrom
ISV-7458

Conversation

@BorekZnovustvoritel

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Marek Szymutko <mszymutk@redhat.com>
Comment thread governance.md

## Contributors

Contributors are community members. They help develop this project. All changes

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is it too strict to require Github issues and pre-approvals for ALL changes? Submitting PRs directly for minor changes is pretty common in most communities. Maybe sign-offs should only be required for significant changes.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fair, but this point should have been raised during the review of the doc. These PRs are already merged in several repos.

Also this reflects the internal contributing guidelines we have already merged. Do we also want to change those?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Oops I might have missed the review of the doc, and I didn't see this in the internal contributing guidelines I think...In that case, let's just get this merged to be consistent with all the other repos. Maybe if it becomes a problem in the future, we can discuss (with the team) about adjusting the policy or wording.

@BorekZnovustvoritel
BorekZnovustvoritel merged commit adf9431 into main Sep 9, 2026
5 checks passed
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