-
Notifications
You must be signed in to change notification settings - Fork 252
Docs: Document about roles, people and processes #1230
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
Merged
Merged
Changes from 7 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
44dbcfa
First draft of adding a role and people documentation
tordans 15617bd
Ask ChatGPT to improve the english language
tordans db89cf3
Maintenance: Clarify guidelines, add Kyle
tordans 6cf79d5
Maintenance: Describe how to merge
tordans 30c4774
Improve wording
tordans 50f0ff7
fix typo
tyrasd 80cb9e8
Docs: Add section on assigning roles
tordans 81464b6
Docs: Add "assign roles" to maintainer role
tordans 817a642
Docs: Update assign roles section
tordans 4e1a225
Docs: Add github labels
tordans 75c556f
rename role to co-maintainers
tyrasd 7de48de
clarify which PRs co-maintainers can merge
tyrasd 8c2d95b
update to "co-maintainer" role name in rest of doc
tyrasd 4d4c6e5
Rename MAINTENANCE.md to GOVERNANCE.md
tyrasd 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,80 @@ | ||
| # Roles, People, and Processes for Maintaining the Tagging Schema | ||
|
|
||
| This document outlines how this project is maintained. | ||
|
|
||
| ## Roles & People | ||
|
|
||
| ### Maintainer Role | ||
|
|
||
| [Martin](https://github.com/tyrasd) maintains this project as part of his work on the iD Editor project. | ||
|
|
||
| The maintainer role includes: | ||
| - Having the final say in decisions for the tagging schema. | ||
| - Creating releases. | ||
| - Updating dependencies. | ||
|
|
||
| and all the following roles. | ||
|
|
||
| ### Contributor Role | ||
|
tyrasd marked this conversation as resolved.
Outdated
|
||
|
|
||
| - [Kyle](https://github.com/k-yle) contributes to this project as a volunteer. | ||
| - [Tobias](https://github.com/tordans) contributes to this project as a volunteer. | ||
|
|
||
| The contributor role includes: | ||
| - Reviewing PRs. | ||
| - Merging "clear-cut" PRs. | ||
|
tyrasd marked this conversation as resolved.
Outdated
|
||
|
|
||
| and all the following roles. | ||
|
|
||
| ### Triage Role | ||
|
|
||
| (No one has the triage role at the moment. Please reach out if you want to help.) | ||
|
|
||
| The triage role includes: | ||
| - Helping to clarify issues and PRs. | ||
|
tyrasd marked this conversation as resolved.
Outdated
|
||
| - Closing issues as duplicates or not planned. | ||
|
|
||
| ### Contributors | ||
|
tyrasd marked this conversation as resolved.
|
||
|
|
||
| To all contributors, thank you so much for your support! ❤ Especially for: | ||
| - Suggesting new presets and fields or updates to the repository. | ||
| - Researching and helping with issues and PRs. | ||
| - Translating the tagging schema. | ||
|
|
||
| Code contributions: [Check this complete list of contributors on GitHub](https://github.com/openstreetmap/id-tagging-schema/graphs/contributors). | ||
|
|
||
| ## Processes | ||
|
|
||
| ### PR Reviews and Merges | ||
|
|
||
| - PRs need approval from two people: the author and one or more contributors before being merged. | ||
| - Non-"clear-cut" changes need to be merged by the maintainer. | ||
| - We might revert merges later if necessary. | ||
|
|
||
| **What is a clear-cut change?** | ||
|
|
||
| - No or minimal controversial discussion on the change. | ||
| - Coding and contribution [guidelines](./GUIDELINES.md) are met. | ||
|
|
||
| **How to merge…** | ||
|
|
||
| - Usually squash merge PRs to make the history simpler | ||
| - Give the merge a meaningful description of the change | ||
| - Add labels to the PR to simplify creating the changelog: `new-*`, `add-*`, `bug` | ||
|
|
||
| ### Releases | ||
|
|
||
| There is no set release schedule at the moment, but releases usually occur every other month. After this project is released, the projects that rely on the data need to update and release as well. | ||
|
|
||
| ### Assigning roles | ||
|
|
||
| - The maintenance role is assigned by the OSMF. Historically the maintainer of the iD Editor also maintained this project. | ||
|
tordans marked this conversation as resolved.
Outdated
|
||
| - Contributor and triage roles are assigned by the current maintainer of the repository. | ||
|
|
||
| ## Previous Maintainers | ||
|
|
||
| Many thanks to all previous maintainers! ❤ | ||
|
|
||
| - [Milos](https://github.com/mbrzakovic) | ||
| - [Quincy](https://github.com/quincylvania) | ||
| - [Bryan](https://github.com/bhousel) | ||
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.