2021-02-05 18:09:44 +00:00
<!--
When creating a new cop that could be applied to multiple applications,
we encourage you to add it to https://gitlab.com/gitlab-org/gitlab-styles gem.
-->
2020-01-23 18:08:53 +00:00
## Description of the proposal
<!--
Please describe the proposal and add a link to the source (for example, http://www.betterspecs.org/).
-->
### Check-list
- [ ] Make sure this MR enables a static analysis check rule for new usage but
2021-12-06 21:10:14 +00:00
ignores current offenses.
- [ ] Mention this proposal in the relevant Slack channels (e.g. `#development` , `#backend` , `#frontend` ).
2020-01-23 18:08:53 +00:00
- [ ] If there is a choice to make between two potential styles, set up an emoji vote in the MR:
- CHOICE_A: :a:
- CHOICE_B: :b:
2021-12-06 21:10:14 +00:00
- Vote for both choices, so they are visible to others.
- [ ] The MR doesn't have significant objections, and is getting a majority of :+1: vs :-1: (remember that [we don't need to reach a consensus ](https://about.gitlab.com/handbook/values/#collaboration-is-not-consensus )).
- [ ] (If applicable) One style is getting a majority of vote (compared to the other choice).
- [ ] (If applicable) Update the MR with the chosen style.
2020-03-18 15:09:45 +00:00
- [ ] Create a follow-up issue to fix the current offenses as a separate iteration: ISSUE_LINK
2021-12-06 21:10:14 +00:00
- [ ] Follow the [review process ](https://docs.gitlab.com/ee/development/code_review.html ) as usual.
2020-01-23 18:08:53 +00:00
- [ ] Once approved and merged by a maintainer, mention it again:
2021-12-06 21:10:14 +00:00
- [ ] In the relevant Slack channels (e.g. `#development` , `#backend` , `#frontend` ).
- [ ] (Optional depending on the impact of the change) In the Engineering Week in Review.
2020-01-23 18:08:53 +00:00
2020-10-12 09:08:38 +00:00
/label ~"Engineering Productivity" ~"development guidelines" ~"static code analysis"
2020-01-23 18:08:53 +00:00
/cc @gitlab -org/maintainers/rails-backend
