Skip to content

Commit 618894b

Browse files
Rename can_admin? -> can_perform_gem_admin?
This variant feels cleaner to me. Hopefully others will agree.
1 parent 30706cf commit 618894b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

app/models/gem_permissions.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def can_manage_owners?
1818
role_granted?(:owner)
1919
end
2020

21-
def can_admin?
21+
def can_perform_gem_admin?
2222
role_granted?(:admin)
2323
end
2424

app/policies/rubygem_policy.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,19 @@ def create?
1313
end
1414

1515
def configure_oidc?
16-
gem_permissions.can_admin? || deny
16+
gem_permissions.can_perform_gem_admin? || deny
1717
end
1818

1919
def configure_trusted_publishers?
20-
gem_permissions.can_admin? || deny
20+
gem_permissions.can_perform_gem_admin? || deny
2121
end
2222

2323
def show_events?
2424
gem_permissions.can_push? || deny
2525
end
2626

2727
def show_unconfirmed_ownerships?
28-
gem_permissions.can_admin? || deny
28+
gem_permissions.can_perform_gem_admin? || deny
2929
end
3030

3131
def add_owner?

0 commit comments

Comments
 (0)