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 riaankl to toc admin #338

Merged
merged 2 commits into from
Dec 22, 2023

Conversation

riaankleinhans
Copy link
Collaborator

In support to toc /tag work

Signed-off-by: Riaan Kleinhans <[email protected]>
Copy link

clowarden bot commented Dec 21, 2023

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • team[cncf-toc]: riaankl must be an organization member to be a maintainer

For more details about the configuration files format please see the documentation.

🔺 These errors must be addressed before this PR can be merged 🔺

@cynthia-sg
Copy link
Contributor

Hi @Riaankl

Please note that CLOWarden expects case sensitive GitHub usernames. In your case, you should use Riaankl.

Signed-off-by: Riaan Kleinhans <[email protected]>
Copy link

clowarden bot commented Dec 22, 2023

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • Riaankl is now a maintainer of team cncf-toc

🔸 Please review the changes detected as they will be applied immediately once this PR is merged 🔸

@riaankleinhans
Copy link
Collaborator Author

Hi @Riaankl

Please note that CLOWarden expects case sensitive GitHub usernames. In your case, you should use Riaankl.

Thanks, that clarify and now things work.

@riaankleinhans riaankleinhans added the ok-to-test Apply if PR is validated safe to test label Dec 22, 2023
@riaankleinhans riaankleinhans requested a review from jeefy December 22, 2023 12:26
@RobertKielty RobertKielty merged commit ee77fa7 into cncf:main Dec 22, 2023
4 of 5 checks passed
Copy link

clowarden bot commented Dec 22, 2023

Reconciliation completed

✅ The reconciliation completed successfully and all changes have been applied across the services!

Changes applied

Github

  • Riaankl is now a maintainer of team cncf-toc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Apply if PR is validated safe to test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants