Skip to content

EDITORS.md: list of current and former specification editors #4022

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 8 commits into from
Aug 22, 2024

Conversation

ralfhandl
Copy link
Contributor

@ralfhandl ralfhandl commented Aug 15, 2024

  • Create EDITORS.md
  • Use it in BUILD.sh for 3.0.4 and 3.1.1

@ralfhandl ralfhandl requested a review from a team as a code owner August 15, 2024 08:40
@ralfhandl ralfhandl added this to the v3.0.4 milestone Aug 15, 2024
Copy link
Member

@earth2marsh earth2marsh left a comment

Choose a reason for hiding this comment

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

Historically this has been the TSC current and former membership list. Perhaps we should make that explicit? Consider whether "contributors.md" is a more appropriate place to recognize the impact that people like Henry make and/or carve out a different section to explain non-TSC maintainer roles?

@ralfhandl
Copy link
Contributor Author

We currently have

  • MAINTAINERS.md: both TSC membership and OAS authors
  • CONTRIBUTORS.md: people with more than 25 pull requests

Now TSC members and OAS authors are becoming non-identical groups of people and we may need three documents.

How about adding an EDITORS.md and use that (from now on) in the build script?

@ralfhandl ralfhandl marked this pull request as draft August 15, 2024 15:19
@ralfhandl ralfhandl marked this pull request as ready for review August 15, 2024 15:29
@ralfhandl ralfhandl requested a review from a team as a code owner August 15, 2024 15:29
@ralfhandl ralfhandl requested a review from earth2marsh August 15, 2024 15:31
@ralfhandl ralfhandl requested a review from a team August 15, 2024 19:53
earth2marsh
earth2marsh previously approved these changes Aug 15, 2024
Copy link
Member

@earth2marsh earth2marsh left a comment

Choose a reason for hiding this comment

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

Thanks for the changes--this looks good to me (other than the minor suggested fix). Ideally we can get another +1, but if not, let's keep moving!

@ralfhandl ralfhandl changed the title MAINTAINERS.md: list specification editors EDITORS.md: list of current and former specification editors Aug 16, 2024
@lornajane lornajane merged commit f8114db into OAI:main Aug 22, 2024
2 checks passed
@ralfhandl ralfhandl deleted the main-update-maintainers branch August 22, 2024 16:24
@earth2marsh
Copy link
Member

Regarding the discussion in today's call, @ralfhandl I believe we can rename EDITORS.md > TSC.md without issue, as the openapis.org governance menu links to the MAINTAINERS.md (and fun fact, to the old branch name, not main!). So once we do the rename, we can work to update the website asynchronously.

If that makes sense to you, LMK, and I'll make an issue and submit that change!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants