Follow-up from "Refactor BranchRules::BaseService to handle common pattern"

The following discussion from !145008 (merged) should be addressed:

  • @ghinfey started a discussion: (+2 comments)

    We are not skipping auth here. Is this inconsistent with the update service?

We should add the ability to skip_authorization to all the ProtectedBranches services. We call these services from other services and mutations that have already authorized the action so we don't need to authorize again.

Assignee Loading
Time tracking Loading