Skip to content

Docs: Document about roles, people and processes#1230

Merged
tyrasd merged 14 commits intoopenstreetmap:mainfrom
tordans:guidelines-roles
Aug 7, 2024
Merged

Docs: Document about roles, people and processes#1230
tyrasd merged 14 commits intoopenstreetmap:mainfrom
tordans:guidelines-roles

Conversation

@tordans
Copy link
Copy Markdown
Collaborator

@tordans tordans commented May 24, 2024

@tyrasd following our conversation on the last iD community call I created this draft of a separate document that gives more visibility on who is doing what in this repository and how some of the core processes are strucutres.

This is not as fleshed out as #1229, especially the process part. But a good way to improve this will be using it and changing it over time, I think.

In this document I changed my role to the contributor role which I still offer to do.


Parts of this are inspired by https://github.com/streetcomplete/StreetComplete/blob/master/AUTHORS.md – thanks a lot

@tordans tordans requested a review from tyrasd May 24, 2024 05:11
@tordans tordans changed the title Docs: Document about role and people Docs: Document about roles, people and processes May 24, 2024
@github-actions
Copy link
Copy Markdown

🍱 You can preview the tagging presets of this pull request here.

@tordans
Copy link
Copy Markdown
Collaborator Author

tordans commented May 26, 2024

In https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/creating-a-default-community-health-file#supported-file-types the suggested name for this file is GOVERNANCE.md

A GOVERNANCE file lets people know about how your project is governed. For example, it might discuss project roles and how decisions are made.

I think this fits what we are trying to do here so I think we should use this name.

Comment thread MAINTENANCE.md Outdated

### PR Reviews and Merges

- PRs need approval from two people: the author and one more contributors before being merged.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- PRs need approval from two people: the author and one more contributors before being merged.
- PRs need approval from two people: the author and one or more contributors before being merged.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There are other ways to amend this. BTW, is "contributor" requiring any experience/presence? Past accepted PR? It is kind of unclear

Copy link
Copy Markdown
Collaborator Author

@tordans tordans Jun 5, 2024

Choose a reason for hiding this comment

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

BTW, is "contributor" requiring any experience/presence? Past accepted PR? It is kind of unclear

I just added a very high level section to handle the question how the defined roles are assigned https://github.com/openstreetmap/id-tagging-schema/pull/1230/files#diff-87a0c4623bffa64c60b62a39eeab1c40bd0f329cebc177f5e4492555fb5d42aeR68-R73
This does not go into details on which criteria are used to make this decision which is IMO fine because in the absence of published criteria it is clear IMO that the person responsible for the step will have to make a judgement decision.

Ping @tyrasd

Comment thread MAINTENANCE.md Outdated
tordans and others added 3 commits June 5, 2024 19:33
Co-authored-by: Mateusz Konieczny <matkoniecz@gmail.com>
Comment thread MAINTENANCE.md Outdated
Comment thread MAINTENANCE.md
Comment thread MAINTENANCE.md Outdated
Comment thread MAINTENANCE.md Outdated
Comment thread MAINTENANCE.md
@matkoniecz
Copy link
Copy Markdown
Collaborator

matkoniecz commented Jun 6, 2024

Just to clarify: all comments I made here are not statement/request/official communication of any organization. And it is just my regular, hopefully useful, nitpicking/debugging.

Comment thread MAINTENANCE.md Outdated
tyrasd and others added 3 commits August 7, 2024 10:56
to avoid confusion with "regular" contributors, and to better represent the great value these people add to the project

also, a few other small amendments to clarify the roles
Co-authored-by: Tobias <t@tobiasjordans.de>
@tyrasd tyrasd merged commit af95f5e into openstreetmap:main Aug 7, 2024
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