ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
canvas-lms/.rubocop.yml

281 lines
5.9 KiB
YAML
Raw Normal View History

add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
require:
- rubocop-rspec
# this odd relative path is so that rubocop works when run without "bundle
# exec", such as from most editors/IDEs.
- ./gems/rubocop-canvas/lib/rubocop_canvas
deterministic described classes, refs SD-1969 This cop encourages explit require_dependency calls for ambiguous nested constants in specs. What does that mean? Consider: module Analytics describe Assignments do Depending on what has been required and/or autoloaded, `Assignments` could either resolve to `Analytics::Assignments`, or just the top-level `Assignments`. This is a cause of flickering failures and test-queue woes. This example should either have an explicit `require_dependency` call, or get rid of the module and just do `describe Analytics::Assignments`` Correct all existing ones in the codebase, except for a few ActiveRecord-related ones (the auto-correct isn't quite perfect, i.e. it assumes the file path will be `const_name.underscore`, which is no bueno for things like ActiveRecord::ConnectionAdapters::PostgreSQLAdapter) Test plan: n/a, specs Change-Id: Ic24bf3e0f547ca11c46887d4af92804da091912a Reviewed-on: https://gerrit.instructure.com/98752 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-01-04 08:52:49 +08:00
AllCops:
TargetRubyVersion: 2.3
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
# our style changes: disabling style rules we aren't interested in
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/AlignParameters:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
turn off rubocop rule for ElseAlignment Change-Id: I3278edf4da55ddb15586937a752e98674db468fa Reviewed-on: https://gerrit.instructure.com/140633 Reviewed-by: Shahbaz Javeed <sjaveed@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com> Tested-by: Jenkins
2018-02-13 02:55:16 +08:00
Layout/ElseAlignment:
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLines:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundAccessModifier:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundArguments:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundBlockBody:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundClassBody:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundMethodBody:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/EmptyLinesAroundModuleBody:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/IndentHash:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/IndentationConsistency:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/IndentationWidth:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/MultilineOperationIndentation:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceAfterColon:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceAfterComma:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceAroundEqualsInParameterDefault:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceAroundOperators:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceBeforeBlockBraces:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceBeforeFirstArg:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceInLambdaLiteral:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceInsideArrayLiteralBrackets:
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceInsideBlockBraces:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Layout/SpaceInsideHashLiteralBraces:
Enabled: false
Layout/SpaceInsideReferenceBrackets:
Enabled: false
Layout/TrailingBlankLines:
Enabled: false
Layout/TrailingWhitespace:
Enabled: false
disable Style/FormatStringToken rubocop test plan: * N/A Change-Id: I0c607f5c33742b824738dd423433c8111c54b3e2 Reviewed-on: https://gerrit.instructure.com/142460 Reviewed-by: Jacob Fugal <jacob@instructure.com> Tested-by: Jenkins Product-Review: Jeremy Neander <jneander@instructure.com> QA-Review: Jeremy Neander <jneander@instructure.com>
2018-03-03 03:36:50 +08:00
Style/FormatStringToken:
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Style/StringLiterals:
Enabled: false
Style/SignalException:
Enabled: false
Style/NumericLiterals:
Enabled: false
Style/BracesAroundHashParameters:
Enabled: false
Style/PercentLiteralDelimiters:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Style/Documentation:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Style/ClassAndModuleChildren:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Style/RegexpLiteral:
Enabled: false
Style/GuardClause:
Enabled: false
Style/RedundantSelf:
add some ruby linting for jenkins to take advantage of add migration lint as rubocop cops create frozen constant linter find_ids datafixup lint send_later lint lint send_later in predeploys add freeze_constant cop to default config don't forget to include rubocop when running get the rubocop runner into the script lint for wrong algorithm name lint primary key lint remove_column in predeploys get rubocop output as parsed json diff munging for gergich disable a few style cops tweak rubocop setup to allow IDE plugins to work get gergich comment format right shell out to gergich if we're in jenkins-land Change-Id: I6eecc8d8ede17a755c9d9a86121c3658776de9cd Reviewed-on: https://gerrit.instructure.com/51755 Tested-by: Jenkins Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-05 08:49:25 +08:00
Enabled: false
Style/IfUnlessModifier:
Enabled: false
Style/WordArray:
Enabled: false
Style/PercentQLiterals:
Enabled: false
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Style/DoubleNegation:
Enabled: false
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
Style/TrailingCommaInArguments:
Enabled: false
Style/TrailingCommaInLiteral:
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Enabled: false
upgrade rubocop and rubocop-rspec to latest closes CNVS-34657 test plan: rubocop should still work Change-Id: Ibdd340dd7ba516639ee5b7225449422b324bd33a Reviewed-on: https://gerrit.instructure.com/100831 Tested-by: Jenkins Reviewed-by: Tyler Pickett <tpickett@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-02-01 02:26:34 +08:00
Style/MethodCallWithoutArgsParentheses:
Enabled: false
Style/MethodCallWithArgsParentheses:
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Enabled: false
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Layout/DotPosition:
re-enable dot position cop, but configured for end-of-line beginning of line can be dangerous when you copy/paste into irb, because it's not greedy, so will execute whatever looks like the first complete statement. this may be an AR query that hasn't been very restricted yet, accidentally destroying the database Change-Id: I878e9e364b84ae57d79ea3185d3bddf3e6d56d8a Reviewed-on: https://gerrit.instructure.com/133054 Tested-by: Jenkins Reviewed-by: Rob Orton <rob@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2017-11-17 02:15:36 +08:00
Enabled: true
EnforcedStyle: trailing
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Layout/AlignHash:
tone down some noisy style complaints Change-Id: I13490195e05df00d58c190761711fe6002b7d6cd Reviewed-on: https://gerrit.instructure.com/52310 Reviewed-by: James Williams <jamesw@instructure.com> Tested-by: Jenkins Product-Review: Jeremy Stanley <jeremy@instructure.com> QA-Review: Jeremy Stanley <jeremy@instructure.com>
2015-04-16 04:49:06 +08:00
Enabled: false
Disable lambda style checking cop The cop wants to see stabbies used for single-line lambdas, and the lambda method for mutli-line lambdas. This is dumb cause stabby lambdas should be used everywhere! Change-Id: I6e9c948688b7192a34a0cff37fe1d1cadcbd047f Reviewed-on: https://gerrit.instructure.com/52877 Tested-by: Jenkins Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Benjamin Porter <bporter@instructure.com> QA-Review: Benjamin Porter <bporter@instructure.com>
2015-04-28 00:23:45 +08:00
Style/Lambda:
Enabled: false
turn off Rubocop::Cop::Style::WhileUntilModifier The modifier while/until construction should not be preferred. The style guide has been updated to reflect this: https://gollum.instructure.com/Ruby-style-guidelines#While/Until Change-Id: Ib1b8b3cf29d52729ba9d3f9066554fc421d47111 Reviewed-on: https://gerrit.instructure.com/52997 Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Strand McCutchen <smccutchen@instructure.com> QA-Review: Strand McCutchen <smccutchen@instructure.com>
2015-04-29 00:27:56 +08:00
Style/WhileUntilModifier:
Enabled: false
Moving ParallelAssignment back to style Change-Id: If7a864309fd97f673c81b1ef2a8a59dcb838fb49 Reviewed-on: https://gerrit.instructure.com/70363 Tested-by: Jenkins Reviewed-by: Jon Jensen <jon@instructure.com> Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2016-01-15 23:44:38 +08:00
Style/ParallelAssignment:
Enabled: false
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
Style/ZeroLengthPredicate:
Enabled: false
Style/NumericPredicate:
Enabled: false
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Naming/VariableNumber:
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
Enabled: false
bump rubocop to 0.50.0 Change-Id: I1aef2dee2089bb61bf220bd8a3640b3c90772188 Reviewed-on: https://gerrit.instructure.com/131135 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-10-30 06:15:03 +08:00
Style/Dir:
Enabled: false
Style/ReturnNil:
Enabled: false
bump rubocop to 0.51.0 Change-Id: I83c54de0e4ec48142c738568d14e54db5766dbbb Reviewed-on: https://gerrit.instructure.com/131136 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-10-30 11:02:24 +08:00
Style/StderrPuts:
Enabled: false
Style/DateTime:
Enabled: false
disable SymbolArray rubocop Change-Id: I70eb38d9097e33236ba04cc013776658304f0b06 Reviewed-on: https://gerrit.instructure.com/144062 Tested-by: Jenkins Reviewed-by: Cameron Matheson <cameron@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-03-20 03:56:20 +08:00
Style/SymbolArray:
Enabled: false
lint: tweak some cops Change-Id: I185cc5dbfaafe64f74a511aac1ab9b94bae38d63 Reviewed-on: https://gerrit.instructure.com/98776 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-01-05 01:51:58 +08:00
# We may want to enable this when we start working toward Ruby 3
Style/FrozenStringLiteralComment:
Enabled: false
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Style/AsciiComments:
Enabled: false
remove curly brace block warning in specs Change-Id: I8f7978adacf41792b3ca3c1e5ea3b09c29b5403e Reviewed-on: https://gerrit.instructure.com/159896 Reviewed-by: Gary Mei <gmei@instructure.com> Reviewed-by: Adrian Packel <apackel@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com> Tested-by: Jenkins
2018-08-07 22:47:36 +08:00
Style/BlockDelimiters:
Enabled: true
Exclude:
- spec/**/*_spec.rb
- spec/shared_examples/**/*.rb
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
spec: get rid of some rubocop-rspec nits * before(:each) is nice since it's often right next to before(:once) * RMessageSpies isn't a thing * don't complain about big blocks (e.g `describe`) Change-Id: I296433cf2478ac3a304a2e20a1a3cd5de6515678 Reviewed-on: https://gerrit.instructure.com/101502 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-02-09 08:12:05 +08:00
# RSpec cops we don't care about
spec: don't nitpick spies vs. message expectations Change-Id: I380b57bc6ee4e7b8e606e5148fb0468031ccee57 Reviewed-on: https://gerrit.instructure.com/101538 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-02-10 01:16:29 +08:00
RSpec/MessageSpies:
Enabled: false
spec: get rid of some rubocop-rspec nits * before(:each) is nice since it's often right next to before(:once) * RMessageSpies isn't a thing * don't complain about big blocks (e.g `describe`) Change-Id: I296433cf2478ac3a304a2e20a1a3cd5de6515678 Reviewed-on: https://gerrit.instructure.com/101502 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-02-09 08:12:05 +08:00
RSpec/HookArgument:
upgrade rubocop and rubocop-rspec to latest closes CNVS-34657 test plan: rubocop should still work Change-Id: Ibdd340dd7ba516639ee5b7225449422b324bd33a Reviewed-on: https://gerrit.instructure.com/100831 Tested-by: Jenkins Reviewed-by: Tyler Pickett <tpickett@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-02-01 02:26:34 +08:00
Enabled: false
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
RSpec/VerifiedDoubles:
Enabled: false
don't care about this cop Change-Id: Ifc88e4990278400c148d2e02326a3658f6b744a6 Reviewed-on: https://gerrit.instructure.com/148093 Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-04-26 00:37:10 +08:00
RSpec/NamedSubject:
Enabled: false
rubocop: I don't care how you phrase this use whichever feels natural Change-Id: If69da65173c7d2a55ff50b9201a7884e8ed5ad96 Reviewed-on: https://gerrit.instructure.com/159475 Tested-by: Jenkins Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-08-03 01:08:40 +08:00
RSpec/NotToNot:
Enabled: false
spec: no more empty example groups remove existing ones and prevent new ones Change-Id: I995bf6c328d229846002a79739beb8eb955a8b1b Reviewed-on: https://gerrit.instructure.com/103950 Tested-by: Jenkins Reviewed-by: Landon Wilkins <lwilkins@instructure.com> Product-Review: Landon Wilkins <lwilkins@instructure.com> QA-Review: Landon Wilkins <lwilkins@instructure.com>
2017-03-04 04:08:13 +08:00
# RSpec cops we care extra about
RSpec/EmptyExampleGroup:
Severity: error
spec: de-dup spec descriptions, fixes SD-2181 also prevent them in the future. our test-queue fork currently has a bug where only the last duplicate spec will run, but this is a good idea anyway most of these are just rewordings, but in some cases delete redundant/ identical specs test plan: specs Change-Id: I8f51be55e750cf87ffa8150822f4b80dc99e1eb4 Reviewed-on: https://gerrit.instructure.com/103993 Tested-by: Jenkins Reviewed-by: Landon Wilkins <lwilkins@instructure.com> Product-Review: Landon Wilkins <lwilkins@instructure.com> QA-Review: Landon Wilkins <lwilkins@instructure.com>
2017-03-04 06:26:51 +08:00
RSpec/RepeatedDescription:
Severity: error
spec: remove ExpectChange and TimesMap some hot takes: - 4.times.map is more readible to Array.new(4).map and has no perf gain in small cases which is the overwhelming use case for specs. This also has no perf impact for app code since it's excluded for specs only. - change { AnonymousOrModerationEvent.count } is more readible than change(AnonymousOrModerationEvent, :count) because the former is the exact code you would see in your app code. This is an RSpec only cop so it has no impact on app code. Change-Id: I6f70b6a9f69a3375cf6f8db27eadd810f25361ca Reviewed-on: https://gerrit.instructure.com/162540 Tested-by: Jenkins Reviewed-by: Spencer Olson <solson@instructure.com> Reviewed-by: Cameron Matheson <cameron@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-08-30 02:30:46 +08:00
RSpec/ExpectChange:
Enabled: false
spec: no more empty example groups remove existing ones and prevent new ones Change-Id: I995bf6c328d229846002a79739beb8eb955a8b1b Reviewed-on: https://gerrit.instructure.com/103950 Tested-by: Jenkins Reviewed-by: Landon Wilkins <lwilkins@instructure.com> Product-Review: Landon Wilkins <lwilkins@instructure.com> QA-Review: Landon Wilkins <lwilkins@instructure.com>
2017-03-04 04:08:13 +08:00
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
# this isn't good for us because of how we pin dependencies
Bundler/OrderedGems:
Enabled: false
bump rubocop to 0.51.0 Change-Id: I83c54de0e4ec48142c738568d14e54db5766dbbb Reviewed-on: https://gerrit.instructure.com/131136 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-10-30 11:02:24 +08:00
Gemspec/OrderedDependencies:
Enabled: false
bump rubocop Change-Id: Ic510fa4a6b34227cf8c6b01bdeea98d9325d1a0d Reviewed-on: https://gerrit.instructure.com/136139 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-12-20 04:08:55 +08:00
Gemspec/RequiredRubyVersion:
Enabled: false
rubocop: remove abc size and changes to severities This commit removes the ostentatious AbcSize metric from rubocop, and changes the following severities: - Style/RescueModifier from convention to warning - Lint/AmbiguousRegexpLiteral from warning to convention - Performance/Detect from convention to warning Change-Id: I6737b4feafaab2d4f94a44cf5f750e8c1779ca38 Reviewed-on: https://gerrit.instructure.com/52220 Reviewed-by: Cameron Matheson <cameron@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:47:24 +08:00
# Rails style changes
disable Migration/ModelBehavior rubocop Change-Id: I68dc9f454d31bf4be75e7fba4dea6d0af75da10c Reviewed-on: https://gerrit.instructure.com/144147 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-03-20 23:24:54 +08:00
Migration/ModelBehavior:
Enabled: false
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
Rails:
Enabled: true
make rlint handle Time.now.utc correctly Change-Id: If2facabedd06c8f16466eebd045988a401834ea1 Reviewed-on: https://gerrit.instructure.com/52036 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-14 00:16:46 +08:00
Rails/TimeZone:
Enabled: false
gergich tweaks * pass additional arguments to script/rlint --heavy -- --additional-args * disable a rails cop * lower style cop to info severity Change-Id: I5cbc94bfc503ae48c635ccb77f66b60f564e1b08 Reviewed-on: https://gerrit.instructure.com/52867 Tested-by: Jenkins Reviewed-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2015-04-27 23:39:18 +08:00
# accessors are often defined in terms of read_attribute
Rails/ReadWriteAttribute:
Enabled: false
bump rubocop Change-Id: Ia8b0c7ec057ae44fb68aa4c2880587762285345a Reviewed-on: https://gerrit.instructure.com/120803 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2017-07-28 04:52:39 +08:00
# we never bothered creating an ApplicationRecord
Rails/ApplicationRecord:
Enabled: false
bump rubocop to 0.50.0 Change-Id: I1aef2dee2089bb61bf220bd8a3640b3c90772188 Reviewed-on: https://gerrit.instructure.com/131135 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-10-30 06:15:03 +08:00
Rails/HasManyOrHasOneDependent:
Enabled: false
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
# It whines about update_all too much, which we use a lot specifically to
# bypass validations and any other AR-ness
Rails/SkipsModelValidations:
Enabled: false
better conditional configuration for rubocop and also remove a useless newline linter Change-Id: I875ed410bc5b54484af503312f75b7021323060a Reviewed-on: https://gerrit.instructure.com/51988 Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 22:43:59 +08:00
rubocop: remove abc size and changes to severities This commit removes the ostentatious AbcSize metric from rubocop, and changes the following severities: - Style/RescueModifier from convention to warning - Lint/AmbiguousRegexpLiteral from warning to convention - Performance/Detect from convention to warning Change-Id: I6737b4feafaab2d4f94a44cf5f750e8c1779ca38 Reviewed-on: https://gerrit.instructure.com/52220 Reviewed-by: Cameron Matheson <cameron@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:47:24 +08:00
# Lint changes
Lint/AmbiguousRegexpLiteral:
Severity: convention
ignore AmbiguousBlockAssociation for specs also fix a typo in .eslintignore caused by cae5d84950 motivation: https://github.com/bbatsov/rubocop/issues/4222#issuecomment-290655562 this is a deliberate idiom in rspec, so let's update .robocop.yml to match rubocop's own config closes: GRADE-841 test plan: specs pass Change-Id: I534eabe48fb227b6fa7b425502e645f22b1a51f3 Reviewed-on: https://gerrit.instructure.com/140424 Tested-by: Jenkins Reviewed-by: Keith T. Garner <kgarner@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-02-08 23:26:27 +08:00
Lint/AmbiguousBlockAssociation:
Exclude:
ignore EnsureSpecExtension for shared_examples Change-Id: I011dc7a8b034d41a82f1c2ddfa69124ca6787ff8 Reviewed-on: https://gerrit.instructure.com/151698 Tested-by: Jenkins Reviewed-by: Jeremy Neander <jneander@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-05-30 01:05:34 +08:00
- spec/**/*
tone down some noisy style complaints Change-Id: I13490195e05df00d58c190761711fe6002b7d6cd Reviewed-on: https://gerrit.instructure.com/52310 Reviewed-by: James Williams <jamesw@instructure.com> Tested-by: Jenkins Product-Review: Jeremy Stanley <jeremy@instructure.com> QA-Review: Jeremy Stanley <jeremy@instructure.com>
2015-04-16 04:49:06 +08:00
Lint/UselessAssignment:
Severity: convention
Rubocop: Flag debuggers and prys as errors Change-Id: I9463bf8e5e725ef51f4924b97451bc378b85216e Reviewed-on: https://gerrit.instructure.com/52825 Reviewed-by: Ethan Vizitei <evizitei@instructure.com> Tested-by: Jenkins Product-Review: Benjamin Porter <bporter@instructure.com> QA-Review: Benjamin Porter <bporter@instructure.com>
2015-04-25 03:27:58 +08:00
Lint/Debugger:
Severity: error
gergich tweaks * pass additional arguments to script/rlint --heavy -- --additional-args * disable a rails cop * lower style cop to info severity Change-Id: I5cbc94bfc503ae48c635ccb77f66b60f564e1b08 Reviewed-on: https://gerrit.instructure.com/52867 Tested-by: Jenkins Reviewed-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2015-04-27 23:39:18 +08:00
Lint/EndAlignment:
change rubocop's end alignment param to be the one we used most often Change-Id: I0ed597588f48d2d3f57111c5a409b1304ebab666 Reviewed-on: https://gerrit.instructure.com/143446 Reviewed-by: Rob Orton <rob@instructure.com> Product-Review: Rob Orton <rob@instructure.com> QA-Review: Rob Orton <rob@instructure.com> Tested-by: Jenkins
2018-03-14 03:15:16 +08:00
EnforcedStyleAlignWith: variable
gergich tweaks * pass additional arguments to script/rlint --heavy -- --additional-args * disable a rails cop * lower style cop to info severity Change-Id: I5cbc94bfc503ae48c635ccb77f66b60f564e1b08 Reviewed-on: https://gerrit.instructure.com/52867 Tested-by: Jenkins Reviewed-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2015-04-27 23:39:18 +08:00
Severity: convention
rubocop: remove abc size and changes to severities This commit removes the ostentatious AbcSize metric from rubocop, and changes the following severities: - Style/RescueModifier from convention to warning - Lint/AmbiguousRegexpLiteral from warning to convention - Performance/Detect from convention to warning Change-Id: I6737b4feafaab2d4f94a44cf5f750e8c1779ca38 Reviewed-on: https://gerrit.instructure.com/52220 Reviewed-by: Cameron Matheson <cameron@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:47:24 +08:00
# Performance changes
Performance/Detect:
Severity: warning
spec: remove ExpectChange and TimesMap some hot takes: - 4.times.map is more readible to Array.new(4).map and has no perf gain in small cases which is the overwhelming use case for specs. This also has no perf impact for app code since it's excluded for specs only. - change { AnonymousOrModerationEvent.count } is more readible than change(AnonymousOrModerationEvent, :count) because the former is the exact code you would see in your app code. This is an RSpec only cop so it has no impact on app code. Change-Id: I6f70b6a9f69a3375cf6f8db27eadd810f25361ca Reviewed-on: https://gerrit.instructure.com/162540 Tested-by: Jenkins Reviewed-by: Spencer Olson <solson@instructure.com> Reviewed-by: Cameron Matheson <cameron@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-08-30 02:30:46 +08:00
Performance/TimesMap:
Exclude:
- spec/**/*
rubocop: remove abc size and changes to severities This commit removes the ostentatious AbcSize metric from rubocop, and changes the following severities: - Style/RescueModifier from convention to warning - Lint/AmbiguousRegexpLiteral from warning to convention - Performance/Detect from convention to warning Change-Id: I6737b4feafaab2d4f94a44cf5f750e8c1779ca38 Reviewed-on: https://gerrit.instructure.com/52220 Reviewed-by: Cameron Matheson <cameron@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:47:24 +08:00
better conditional configuration for rubocop and also remove a useless newline linter Change-Id: I875ed410bc5b54484af503312f75b7021323060a Reviewed-on: https://gerrit.instructure.com/51988 Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 22:43:59 +08:00
# these need better configuration than the default:
Style/AndOr:
EnforcedStyle: conditionals
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
Style/RescueModifier:
Severity: warning
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Layout/MultilineMethodCallIndentation:
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
EnforcedStyle: indented
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Layout/IndentArray:
upgrade rubocop to 0.46.0 and rubocop-rspec to 1.8.0 note: Astrolabe::Node is now just Rubocop::Node closes CNVS-34077 test plan: rubocop should still lint correctly Change-Id: Id5feb1700ad78cc0b34c6cb90d2a75320b210138 Reviewed-on: https://gerrit.instructure.com/98512 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2016-12-28 14:17:05 +08:00
EnforcedStyle: consistent
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
# these don't make sense pre-Rails 4
Rails/FindBy:
Enabled: false
back off rails 4 rubocop Change-Id: I6b833430f224051c93c2b36803bc4cfc1ce2376c Reviewed-on: https://gerrit.instructure.com/51986 Reviewed-by: Matthew Wheeler <mwheeler@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 21:37:58 +08:00
Rails/ActionFilter:
Enabled: false
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
add rubocop rules for rspec nesting and http arguments This disables the HttpPositionalArguments rubocop rule, which is invalid for versions of Ruby prior to Ruby 5. This also sets the limit for nested example groups in RSpec to 3, which is a more reasonable limit than the default of 2. test plan: * verify Jenkins passes Change-Id: I8f2badb369d6932923aa4c096c9b9014be19805a Reviewed-on: https://gerrit.instructure.com/100198 Tested-by: Jenkins Reviewed-by: Rob Orton <rob@instructure.com> QA-Review: Jeremy Neander <jneander@instructure.com> Product-Review: Jeremy Neander <jneander@instructure.com>
2017-01-21 22:18:29 +08:00
# these are invalid pre-Rails 5
Rails/HttpPositionalArguments:
Enabled: false
upgrade rubocop and rubocop-rspec to latest closes CNVS-34657 test plan: rubocop should still work Change-Id: Ibdd340dd7ba516639ee5b7225449422b324bd33a Reviewed-on: https://gerrit.instructure.com/100831 Tested-by: Jenkins Reviewed-by: Tyler Pickett <tpickett@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-02-01 02:26:34 +08:00
# these are invalid pre-Ruby 2.4
Performance/RegexpMatch:
Enabled: false
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
# Things we may want to tighten down later
colorize rlint output Change-Id: I357a15e38c454991efd556491cd8f047a07f701a Reviewed-on: https://gerrit.instructure.com/52008 Reviewed-by: Braden Anderson <braden@instructure.com> Product-Review: Braden Anderson <braden@instructure.com> QA-Review: Braden Anderson <braden@instructure.com> Tested-by: Jenkins
2015-04-11 02:29:31 +08:00
Metrics/AbcSize:
rubocop: remove abc size and changes to severities This commit removes the ostentatious AbcSize metric from rubocop, and changes the following severities: - Style/RescueModifier from convention to warning - Lint/AmbiguousRegexpLiteral from warning to convention - Performance/Detect from convention to warning Change-Id: I6737b4feafaab2d4f94a44cf5f750e8c1779ca38 Reviewed-on: https://gerrit.instructure.com/52220 Reviewed-by: Cameron Matheson <cameron@instructure.com> Reviewed-by: Jeremy Stanley <jeremy@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:47:24 +08:00
Enabled: false
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Metrics/LineLength:
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Max: 160
make gergich a better coworker Change-Id: I9a04d12fc49f6e0095359e5dabbb5f5f18629692 Reviewed-on: https://gerrit.instructure.com/51939 Reviewed-by: Cody Cutrer <cody@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 03:55:31 +08:00
Metrics/MethodLength:
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Max: 100
Metrics/ClassLength:
Enabled: false
Metrics/ModuleLength:
Enabled: false
spec: get rid of some rubocop-rspec nits * before(:each) is nice since it's often right next to before(:once) * RMessageSpies isn't a thing * don't complain about big blocks (e.g `describe`) Change-Id: I296433cf2478ac3a304a2e20a1a3cd5de6515678 Reviewed-on: https://gerrit.instructure.com/101502 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-02-09 08:12:05 +08:00
Metrics/BlockLength:
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Max: 50
spec: get rid of some rubocop-rspec nits * before(:each) is nice since it's often right next to before(:once) * RMessageSpies isn't a thing * don't complain about big blocks (e.g `describe`) Change-Id: I296433cf2478ac3a304a2e20a1a3cd5de6515678 Reviewed-on: https://gerrit.instructure.com/101502 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Jon Jensen <jon@instructure.com> QA-Review: Jon Jensen <jon@instructure.com>
2017-02-09 08:12:05 +08:00
Exclude:
ignore EnsureSpecExtension for shared_examples Change-Id: I011dc7a8b034d41a82f1c2ddfa69124ca6787ff8 Reviewed-on: https://gerrit.instructure.com/151698 Tested-by: Jenkins Reviewed-by: Jeremy Neander <jneander@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-05-30 01:05:34 +08:00
- gems/*/spec/**/*.rb
- gems/plugins/*/spec_canvas/**/*.rb
- spec/**/*.rb
spec: Ignore block length rule in Pact tests We're already ignoring it in RSpec and Selenium tests. And Pact tests inherently have a lot of lines. Change-Id: Ia976224a03426864beb04d885011e982d4c2a60d Reviewed-on: https://gerrit.instructure.com/156583 Reviewed-by: Michael Hargiss <mhargiss@instructure.com> Tested-by: Jenkins Product-Review: Tucker McKnight <tmcknight@instructure.com> QA-Review: Tucker McKnight <tmcknight@instructure.com>
2018-07-10 02:26:04 +08:00
- pact/spec/**/*_spec.rb
colorize rlint output Change-Id: I357a15e38c454991efd556491cd8f047a07f701a Reviewed-on: https://gerrit.instructure.com/52008 Reviewed-by: Braden Anderson <braden@instructure.com> Product-Review: Braden Anderson <braden@instructure.com> QA-Review: Braden Anderson <braden@instructure.com> Tested-by: Jenkins
2015-04-11 02:29:31 +08:00
Metrics/CyclomaticComplexity:
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Max: 28
Metrics/PerceivedComplexity:
Max: 28
back off, gergich Change-Id: Id6b537dbfe139a9e63079d2cc3d04a08ad72c8cd Reviewed-on: https://gerrit.instructure.com/51966 Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 06:29:09 +08:00
RSpec/InstanceVariable:
Enabled: false
RSpec/ExampleWording:
Enabled: false
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
RSpec/ContextWording:
Enabled: false
lint: tweak some cops Change-Id: I185cc5dbfaafe64f74a511aac1ab9b94bae38d63 Reviewed-on: https://gerrit.instructure.com/98776 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-01-05 01:51:58 +08:00
RSpec/ExampleLength:
relax some rubocop nits yes, some of them are kind of sad/bad. but the signal to noise with them on is so bad that we end up ignoring far more useful advice because we ignore gergich completely Change-Id: Ie62b9e4c6459440c7c6cfaa29bae8271be639d77 Reviewed-on: https://gerrit.instructure.com/141069 Reviewed-by: Rob Orton <rob@instructure.com> Tested-by: Jenkins Reviewed-by: Simon Williams <simon@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
2018-02-16 01:25:36 +08:00
Max: 48
spec: increase rubocop allowed rspec nested groups Change-Id: Ice28cad151efa8f58e45b88eb3b11ac248d07c7b Reviewed-on: https://gerrit.instructure.com/100104 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-01-21 00:35:25 +08:00
RSpec/NestedGroups:
upgrade rubocop and rubocop-rspec to latest closes CNVS-34657 test plan: rubocop should still work Change-Id: Ibdd340dd7ba516639ee5b7225449422b324bd33a Reviewed-on: https://gerrit.instructure.com/100831 Tested-by: Jenkins Reviewed-by: Tyler Pickett <tpickett@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-02-01 02:26:34 +08:00
Max: 5
back off, gergich Change-Id: Id6b537dbfe139a9e63079d2cc3d04a08ad72c8cd Reviewed-on: https://gerrit.instructure.com/51966 Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-10 06:29:09 +08:00
RSpec/DescribedClass:
Enabled: false
Plugins UI needs 'dynamic_form' closes CNVS-19816 make sure we have a spec to check rendering of the plugins setting view from now on, and explicitly require 'dynamic_form' in the plugins controller also removes a linter that shouldn't be on for describing classes (or views in this case) TEST PLAN: 1) go configure any plugin 2) it should not explode Change-Id: I32e58a524ddb18b922ba79e20e9e7b9c1f2a8f07 Reviewed-on: https://gerrit.instructure.com/52216 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins QA-Review: Ethan Vizitei <evizitei@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com>
2015-04-15 06:21:54 +08:00
RSpec/DescribeClass:
Enabled: false
add boyscout mode for rlint also back off the hash syntax rule Change-Id: Ib5daec82b013cb5ca3abeec7705f3838cc438d06 Reviewed-on: https://gerrit.instructure.com/52023 Reviewed-by: Jason Madsen <jmadsen@instructure.com> Product-Review: Ethan Vizitei <evizitei@instructure.com> QA-Review: Ethan Vizitei <evizitei@instructure.com> Tested-by: Ethan Vizitei <evizitei@instructure.com>
2015-04-11 04:49:20 +08:00
Style/HashSyntax:
Enabled: false
lint: tweak some cops Change-Id: I185cc5dbfaafe64f74a511aac1ab9b94bae38d63 Reviewed-on: https://gerrit.instructure.com/98776 Reviewed-by: Cody Cutrer <cody@instructure.com> Tested-by: Jenkins Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-01-05 01:51:58 +08:00
RSpec/MultipleExpectations:
Enabled: false
upgrade rubocop and rubocop-rspec to latest closes CNVS-34657 test plan: rubocop should still work Change-Id: Ibdd340dd7ba516639ee5b7225449422b324bd33a Reviewed-on: https://gerrit.instructure.com/100831 Tested-by: Jenkins Reviewed-by: Tyler Pickett <tpickett@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-02-01 02:26:34 +08:00
RSpec/ScatteredSetup:
Enabled: false
update rubocop rubocop changed the namespace on some items. This updates to the new names so that it won't toss out warnings that things have moved. Change-Id: Ie9003672a10adebab983b1b91393cc83a207b3c3 Reviewed-on: https://gerrit.instructure.com/136688 Reviewed-by: Simon Williams <simon@instructure.com> Tested-by: Jenkins Product-Review: Keith T. Garner <kgarner@instructure.com> QA-Review: Keith T. Garner <kgarner@instructure.com>
2018-01-03 01:29:58 +08:00
Style/RescueStandardError:
bump rubocop to 0.50.0 Change-Id: I1aef2dee2089bb61bf220bd8a3640b3c90772188 Reviewed-on: https://gerrit.instructure.com/131135 Tested-by: Jenkins Reviewed-by: Cody Cutrer <cody@instructure.com> Product-Review: Simon Williams <simon@instructure.com> QA-Review: Simon Williams <simon@instructure.com>
2017-10-30 06:15:03 +08:00
Enabled: false
ignore EnsureSpecExtension for shared_examples Change-Id: I011dc7a8b034d41a82f1c2ddfa69124ca6787ff8 Reviewed-on: https://gerrit.instructure.com/151698 Tested-by: Jenkins Reviewed-by: Jeremy Neander <jneander@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
2018-05-30 01:05:34 +08:00
Specs/EnsureSpecExtension:
Enabled: true
Exclude:
- spec/shared_examples/**/*