Commit c7266a46 authored by Achilleas Pipinellis's avatar Achilleas Pipinellis

Merge branch '220374-doc-add-gitlab-styles-process' into 'master'

Add rubocop creation process

Closes #220374

See merge request gitlab-org/gitlab!34754
parents 6b34cb89 3fd91306
...@@ -56,6 +56,16 @@ Additionally, we have a dedicated ...@@ -56,6 +56,16 @@ Additionally, we have a dedicated
[newlines style guide](../newlines_styleguide.md), as well as dedicated [newlines style guide](../newlines_styleguide.md), as well as dedicated
[test-specific style guides and best practices](../testing_guide/index.md). [test-specific style guides and best practices](../testing_guide/index.md).
### Creating new RuboCop cops
Typically it is better for the linting rules to be enforced programmatically as it
reduces the aforementioned [bike-shedding](https://en.wiktionary.org/wiki/bikeshedding).
To that end, we encourage creation of new RuboCop rules in the codebase.
When creating a new cop that could be applied to multiple applications, we encourage you
to add it to our [GitLab Styles](https://gitlab.com/gitlab-org/gitlab-styles) gem.
## Database migrations ## Database migrations
See the dedicated [Database Migrations Style Guide](../migration_style_guide.md). See the dedicated [Database Migrations Style Guide](../migration_style_guide.md).
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment