-
Notifications
You must be signed in to change notification settings - Fork 173
docs: add Technical Charter and restructure governance documentation #1664
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
Open
kriswest
wants to merge
4
commits into
finos:main
Choose a base branch
from
kriswest:update-governance-3
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+156
−31
Open
Changes from all commits
Commits
Show all changes
4 commits
Select commit
Hold shift + click to select a range
08d3f31
chore(governance): update project governance to current FINOS standar…
kriswest 55cf7f0
Merge branch 'main' into update-governance-3
kriswest 7f3fde3
Merge branch 'main' into update-governance-3
andypols 1423228
Merge branch 'main' into update-governance-3
kriswest File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| # GitProxy Technical Charter | ||
|
|
||
| ## Mission | ||
|
|
||
| GitProxy enables organisations — particularly regulated financial institutions and other enterprises — to enforce configurable policies on outgoing Git push operations, ensuring compliance with security, legal, and risk requirements while preserving the standard open source developer experience. | ||
|
|
||
| ## Scope | ||
|
|
||
| The project develops and maintains: | ||
|
|
||
| - The Git HTTP and SSH proxy server | ||
| - The policy engine (processor chain and plugin system) | ||
| - The approval and review workflow | ||
| - The service API and web-based dashboard | ||
| - The command-line interface (CLI) | ||
| - Associated documentation and tooling | ||
|
|
||
| Out of scope: | ||
|
|
||
| - Organisation-specific policy content or plugin implementations | ||
| - Hosting, infrastructure, or operational support for production deployments | ||
|
|
||
| ## Maintainer Structure | ||
|
|
||
| The project community consists of Contributors and Maintainers: | ||
|
|
||
| - **Contributor** — anyone who submits a contribution to the project (code, issues, comments, documentation, media, or any combination). | ||
| - **Maintainer** — a Contributor who, by virtue of their contribution history, has been given write access to project repositories and may merge approved contributions. | ||
| - **Lead Maintainer** — the project's interface with the FINOS team and Board. Responsible for approving quarterly project reports and communicating on behalf of the project. Elected by a vote of the Maintainers. | ||
|
|
||
| The Maintainers collectively serve as the project's technical steering body. The current roster is recorded in [`MAINTAINERS.md`](MAINTAINERS.md). | ||
|
|
||
| ## Decision Making | ||
|
|
||
| Maintainers reach decisions by consensus where possible, and by vote when necessary. The voting process, contribution rules, cross-firm review requirements, and dispute resolution procedures are defined in [`CONTRIBUTING.md`](CONTRIBUTING.md). | ||
|
|
||
| ## Intellectual Property | ||
|
|
||
| - The project is licensed under the [Apache License, Version 2.0](LICENSE). | ||
| - Contributions are governed by the [FINOS IP Policy](https://community.finos.org/assets/files/IP-Policy-fe5925025fc0a57b1cbed64f86b26a73.pdf). | ||
| - All contributors must have a Contributor License Agreement (CLA) on file with FINOS before contributions can be merged. | ||
|
|
||
| ## Amendments | ||
|
|
||
| This charter may be amended by a vote of the Maintainers according to the voting process defined in [`CONTRIBUTING.md`](CONTRIBUTING.md), subject to FINOS Board approval. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
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.
I get the spirit of this statement. However, this is in conflict with the project providing a Docker image for deployment use. I would classify that as "hosting" or "infrastructure" to a degree (firms are of course free to pull in the project from source and build/deploy the app as they see fit). It's not unreasonable for the project to provide some runnable artifact and offer a degree of support (in line with SECURITY.md and responsible disclosure policies).
If the purpose of this is to say "finos/git-proxy the project cannot provide operational support such as support for hosting Git Proxy or required infrastructure to operate Git Proxy", then I'm good with this.