Skip to content

Add GSoC 2026 contributors to the gsoc-contributors team#2455

Open
Kobzol wants to merge 2 commits intorust-lang:mainfrom
Kobzol:gsoc-2026
Open

Add GSoC 2026 contributors to the gsoc-contributors team#2455
Kobzol wants to merge 2 commits intorust-lang:mainfrom
Kobzol:gsoc-2026

Conversation

@Kobzol
Copy link
Copy Markdown
Member

@Kobzol Kobzol commented May 1, 2026

jieyouxu
jieyouxu previously approved these changes May 2, 2026
Copy link
Copy Markdown
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Team changes looks good

Comment thread src/validate.rs
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Question: should this use excluded-people for the zulip group instead? Last recall I changed the validation in #1977

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, I recalled that there was something like this, but I forgot about this field. Yeah, that is better, thanks!

@jieyouxu jieyouxu added needs-team-repo-admin-review This change requires one of the `team-repo-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. labels May 2, 2026
@jieyouxu jieyouxu dismissed their stale review May 2, 2026 09:06

Use excluded-people for not-requiring zulip ID instead

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

Labels

needs-team-repo-admin-review This change requires one of the `team-repo-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants