search

cncf / tag-security

🔐CNCF Security Technical Advisory Group -- secure access, policy control, privacy, auditing, explainability and more!
https://cncf.io/projects
Other
1.99k stars 496 forks source link

Clean-up: Update settings.yml #1232

Closed riaankleinhans closed 3 months ago

riaankleinhans commented 4 months ago

People and permissions have been moved to the cncf/people/config.yml file making permissions in the local settings.yml file redundant.

Proposed updates:

netlify[bot] commented 4 months ago

Deploy Preview for tag-security canceled.

Name Link
Latest commit e0d85a840ffa8b2ec19131b0db7adf6e0bdd0a52
Latest deploy log https://app.netlify.com/sites/tag-security/deploys/65e7cd28366eb800088f8ca9
PushkarJ commented 3 months ago

@Riaankl thank you for the PR. Can you please add @lirantal in the new configuration file? He is the only one missing from the list we are removing.

riaankleinhans commented 3 months ago

@Riaankl thank you for the PR. Can you please add @lirantal in the new configuration file? He is the only one missing from the list we are removing.

No problems @PushkarJ I can add @lirantal , with write permission? Which role does @lirantal have in the TAG?

PushkarJ commented 3 months ago

"Read" permission for @lirantal is what we have today so please do the same

riaankleinhans commented 3 months ago

@PushkarJ I created a PR for adding @lirantal to the config.yml file in cncf/people.

Can you please merge this PR as it is not dependent on the other PR.

Thanks!

lirantal commented 3 months ago

Thanks folks!