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

Added permissions for the project-template repo. #270

Merged
merged 3 commits into from
Aug 8, 2023

Conversation

geekygirldawn
Copy link
Member

@geekygirldawn geekygirldawn commented Aug 8, 2023

Giving myself admin access to the project-template repo.

Confirmation that I should have this access can be found in this issue.

…dmin access and added our TOC Liaisons with write access since I think that will make it possible for me to assign them as reviewers (which I can't now).

Signed-off-by: Dawn M. Foster <dawn@dawnfoster.com>
@clowarden
Copy link

clowarden bot commented Aug 8, 2023

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • error parsing permissions file
      • repositories[100]: missing field name at line 693 column 5

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

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

Signed-off-by: Dawn M. Foster <dawn@dawnfoster.com>
@clowarden
Copy link

clowarden bot commented Aug 8, 2023

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • error parsing permissions file
      • repositories[101]: missing field name at line 708 column 5

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

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

…misunderstanding the order of the file.

Signed-off-by: Dawn M. Foster <dawn@dawnfoster.com>
@clowarden
Copy link

clowarden bot commented Aug 8, 2023

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Github

  • user geekygirldawn role in repository project-template has been updated to admin

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

@RobertKielty RobertKielty requested a review from jeefy August 8, 2023 13:49
@RobertKielty RobertKielty self-assigned this Aug 8, 2023
@RobertKielty RobertKielty added the ok-to-test Apply if PR is validated safe to test label Aug 8, 2023
@RobertKielty RobertKielty merged commit 253c07a into cncf:main Aug 8, 2023
4 of 6 checks passed
@clowarden
Copy link

clowarden bot commented Aug 8, 2023

Reconciliation completed

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

Changes applied

Github

  • user geekygirldawn role in repository project-template has been updated to admin

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.

2 participants