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

SafeSettings removes existing team permission for all the repos #350

Open
sajithvasu opened this issue Dec 5, 2022 · 4 comments
Open

SafeSettings removes existing team permission for all the repos #350

sajithvasu opened this issue Dec 5, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@sajithvasu
Copy link

sajithvasu commented Dec 5, 2022

Github Safesettings app removes the existing teams permissions for all existing repositories within the org when the following mentioned settings are enabled. On the other hand, documentation does not specify removal of any existing permission when controlled through safesettings app. Is this the expected behavior?

settings.yml

repository:
teams:
  - name: <team_name>
    permission: <team_permission>

Our goal is to add specific permissions/teams to all the existing/new repos without impacting existing team permission for all the repos.

@sajithvasu sajithvasu added the bug Something isn't working label Dec 5, 2022
@martinm82
Copy link
Contributor

I have discovered this issue as well and @decyjphr confirmed this here. It would be indeed good to update the existing settings instead of overriding them fully.

@decyjphr
Copy link
Collaborator

decyjphr commented Dec 8, 2022

This fix has been released in 2.0.18

@martinm82
Copy link
Contributor

@decyjphr should we close this issue? Also worth to add a reference to the pull request solving this problem.

@anderssonjohan
Copy link
Contributor

Isn't this contradicting to #251? In my situation, I want to remove all manually added collaborators because it should only be a single source of truth when using this for policy-as-code. Related to fixes in #382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants