Skip to content
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

docs: update maintainers status #512

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Feggah
Copy link
Collaborator

@Feggah Feggah commented Sep 6, 2023

Description of your changes

In order to send the correct signal to the community about the maintainers of this provider, I updated the OWNERS.md file.

I have shifted my career and attention to other projects and technologies, giving me little bandwidth to continue contributing to the project. With that being said, I think it makes sense to remove my Maintainer status to best reflect my absence of contributions in the Crossplane ecosystem.

I also have updated @hasheddan maintainer status to Emeritus Maintainer. I guess it makes sense to follow the same status as in the Crossplane core repository.

If you still want to be in the Maintainer section, let me know and I will revert it @hasheddan.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

N/A

Copy link
Member

@hasheddan hasheddan 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 all of your awesome work here @Feggah! I'll stick around as maintainer for the time-being and do my best to support @dee0sap while we recruit additional maintainers 👍🏻 Feel free to ping when this PR is updated!

@dee0sap
Copy link
Collaborator

dee0sap commented Sep 6, 2023

Thanks @hasheddan

Btw, is it acceptable for there to be more than 1 maintainer from the same company?

I ask because my team would like to add another of its members as a maintainer. ( Subject to approval from maintainers other than myself of course :) )

@Feggah
Copy link
Collaborator Author

Feggah commented Sep 6, 2023

Thanks for all of your awesome work here @Feggah! I'll stick around as maintainer for the time-being and do my best to support @dee0sap while we recruit additional maintainers 👍🏻 Feel free to ping when this PR is updated!

Thanks for the message Dan! I reverted the change to emeritus that I made 🙂

Copy link
Member

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

Btw, is it acceptable for there to be more than 1 maintainer from the same company?

@dee0sap yes!

Thanks @Feggah!

@jbw976
Copy link
Member

jbw976 commented Sep 7, 2023

Btw, is it acceptable for there to be more than 1 maintainer from the same company?

I ask because my team would like to add another of its members as a maintainer. ( Subject to approval from maintainers other than myself of course :) )

Yes that's totally fine! and for future reference, GOVERNANCE.md for crossplane covers these scenarios, so feel free to refer to it and let me know if you have any questions!

In general, the maintainer team has autonomy to update the team for their repo on their own. e.g.:

In general, adding and removing maintainers for a given repo is the responsibility of the existing maintainer team for that repo and therefore does not require approval from the steering committee.

Copy link
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

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

big thanks for all your efforts on the project to date @Feggah! 🙇‍♂️

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.

4 participants