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 .asf.yaml #10767

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Update .asf.yaml #10767

merged 1 commit into from
Jul 25, 2024

Conversation

ajantha-bhat
Copy link
Member

No description provided.

@github-actions github-actions bot added the INFRA label Jul 24, 2024
@@ -49,7 +49,6 @@ github:
- jun-he
- marton-bod
- samarthjain
- findepi
Copy link
Member Author

@ajantha-bhat ajantha-bhat Jul 24, 2024

Choose a reason for hiding this comment

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

@findepi: Congratulations on being committer :)

I would also like to remove the people who are inactive from 6 months or more in iceberg repo from this list to make space for new people who are active and intersted in project management. We can add them back if they are active again. What do you guys think?

Copy link
Member Author

Choose a reason for hiding this comment

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

And @jbonofre: Since you are more active on the release side, I think we should add you here :)

Copy link
Member

Choose a reason for hiding this comment

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

@ajantha-bhat 😄 Do you want to add me in this PR or do you want me to create another PR ?

Copy link
Member

Choose a reason for hiding this comment

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

I agree with the 6 months period. It sounds reasonable to me.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good one @ajantha-bhat, and I'm all in for adding @jbonofre to the list 👍

@findepi
Copy link
Member

findepi commented Jul 24, 2024

@ajantha-bhat this PR probably reduces my access to the repo currently. Would you be willing to merge it once I actually have the committer access. We can test it eg by me merging (which it seems I cannot yet)

@ajantha-bhat
Copy link
Member Author

@ajantha-bhat this PR probably reduces my access to the repo currently. Would you be willing to merge it once I actually have the committer access. We can test it eg by me merging (which it seems I cannot yet)

@findepi: Sure. I assumed we give privileges before announcing on the mailing list. Feel free to merge this PR when you get the access. No hurry.

@findepi findepi merged commit 622c127 into apache:main Jul 25, 2024
49 checks passed
@findepi
Copy link
Member

findepi commented Jul 25, 2024

@ajantha-bhat thanks for your patience. Looks we're all set.
And this is definitely the best first PR to merge :)

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

Successfully merging this pull request may close these issues.

5 participants