<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=435208) </details> <!--IssueSummary end--> While investigating PushRule logic I noticed that the global push rule is defined using a `is_sample` column which does not clearly indicate the intention of the column and also goes against ruby naming norms. We use a column ["push_rules"."is_sample"](https://gitlab.com/gitlab-org/gitlab/-/blob/master/db/structure.sql#L22615) to define if the `PushRule` is `global`. We also have a method [PushRule#global?](https://gitlab.com/gitlab-org/gitlab/-/blob/399901149d67a2f33c623c55e07c273c2f152cfe/ee/app/models/push_rule.rb#L130) that checks if `is_sample` is true. We then have another method [PushRule::global](https://gitlab.com/gitlab-org/gitlab/-/blob/399901149d67a2f33c623c55e07c273c2f152cfe/ee/app/models/push_rule.rb#L59) to fetch the global rules by calling `find_by(is_sample: true)` We should rename the column to `global` so it is clear what this column configures. Once we rename we can remove the `PushRule#global?` method as it will be generated by `ActiveRecord::Base` automatically.