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

Add Yusmen Zabanov as a member of the cloudfoundry org #931

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

georgethebeatle
Copy link
Member

Yusmen is a colleague from SAP who is joining the cf-on-k8s working group

It looks like contributor names were mostly sorted in lexicographical
order (case insensitive) with a few exceptions. This changes fixes the
order without changing the list of contributors in any way
Copy link
Member

@beyhan beyhan left a comment

Choose a reason for hiding this comment

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

I don't find yusmen-zabanov_sap as user and https://github.com/yusmen-zabanov_sap returns 404.

@georgethebeatle
Copy link
Member Author

Fair point @beyhan

I have provided the proper user now, sorry for the confusion

@beyhan beyhan self-requested a review July 24, 2024 11:08
@beyhan
Copy link
Member

beyhan commented Jul 24, 2024

According to https://github.com/cloudfoundry/community/blob/main/toc/ROLES.md#promotion-and-revocation

Two existing Contributors, Reviewers, or Approvers must attest that they meet the criteria by reviewing the PR.

When the above is fulfilled you can merge the pr.

@christopherclark christopherclark self-requested a review July 26, 2024 14:15
@georgethebeatle georgethebeatle merged commit 1c4e32a into main Jul 29, 2024
2 checks passed
@georgethebeatle georgethebeatle deleted the add_yusmen_zabanov branch January 8, 2025 10:14
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