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

Update app-runtime-interfaces buildpacks approvers #719

Merged
merged 2 commits into from
Jan 4, 2024
Merged

Update app-runtime-interfaces buildpacks approvers #719

merged 2 commits into from
Jan 4, 2024

Conversation

brayanhenao
Copy link
Member

Changes

Removal of Ryan Moran

I removed Ryan Moran from all relevant groups, he is no longer part of the Buildpacks Team.

Addition of Brayan Henao, Replacing Ryan Moran in Approvers' Roles

In response to approvers responsibilities left by Ryan Moran's departure, I have taken the initiative to introduce me into some specific teams. This decision is based on the need for at least three approvers in teams where Ryan's absence created a gap. The following exceptions apply:

  • Buildpacks Tooling: Given my active involvement in the v2b release process, particularly within the buildpacks-ci repository and its related counterparts, I am confident in fulfilling the approver role.

  • Buildpacks Docs: I haven't taken up the approver role there yet because I haven't contributed enough to the repositories

For your reference, please review my contributions in the following areas:

@brayanhenao
Copy link
Member Author

I'll tag some approvers from each team:

@brayanhenao
Copy link
Member Author

C.C @Gerg

@Gerg Gerg self-requested a review December 4, 2023 22:27
@Gerg
Copy link
Member

Gerg commented Dec 4, 2023

Per the role revocation policy, @ryanmoran has two weeks to refute this revocation, if they wish.

@Gerg
Copy link
Member

Gerg commented Dec 4, 2023

As for @brayanhenao's promotions: According to the promotion process, a single approval per area is sufficient in these cases because the areas have fewer than 4 approvers each.

@beyhan beyhan added the wg label Dec 5, 2023
@beyhan
Copy link
Member

beyhan commented Dec 11, 2023

Hi @brayanhenao and @Gerg,
should Ryan be removed from the contributors.yml also? If we know that Ryan won't be active in the CF organization we do this to keep the list with the org members clean.

@brayanhenao
Copy link
Member Author

brayanhenao commented Dec 11, 2023

@beyhan I submitted another commit modifying contributors.yml. He will not be active in the CF community

@beyhan
Copy link
Member

beyhan commented Dec 19, 2023

The two week period is over and there was no objection. This could be merged. @Gerg

@Gerg Gerg merged commit 2b212e2 into cloudfoundry:main Jan 4, 2024
2 checks passed
beyhan added a commit that referenced this pull request Jan 12, 2024
ryanmoran has been removed from the cloudfoundry org with
this [pr](#719). The is
still a role assigned to him which fails the org automation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants