- 08 Jul, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 01 Jul, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 30 Jun, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 29 Jun, 2016 3 commits
-
-
Grzegorz Bizon authored
-
Grzegorz Bizon authored
-
Yorick Peterse authored
There are currently two cops for this: * Migration/AddIndex: checks if indexes are added concurrently * Migration/ColumnWithDefault: checks if columns with default values are added in a concurrent manner Both cops are fairly simple and make no attempt at correcting the code as this is quite hard to do (e.g. modifications may need to be applied in various places in the same file). They however should provide enough to catch people ignoring the comments in generated migrations telling them to use add_concurrent_index or add_column_with_default.
-
- 16 Jun, 2016 2 commits
-
-
James Lopez authored
This reverts commit 13e37a3e.
-
James Lopez authored
-
- 10 Jun, 2016 1 commit
-
-
Grzegorz Bizon authored
See #17478
-
- 09 Jun, 2016 1 commit
-
-
Sean McGivern authored
Migrations shouldn't fail RuboCop checks - especially lint checks, such as the nested method check. To avoid changing code in existing migrations, add the magic comment to the top of each of them to skip that file.
-
- 04 Jun, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 03 Jun, 2016 4 commits
-
-
James Lopez authored
This reverts commit 3e991230.
-
Grzegorz Bizon authored
See #17478
-
James Lopez authored
# Conflicts: # app/models/project.rb
-
Grzegorz Bizon authored
See #17478
-
- 01 Jun, 2016 2 commits
-
-
Grzegorz Bizon authored
Do not use for, unless you know exactly why. See #17478
-
Grzegorz Bizon authored
See #17478
-
- 31 May, 2016 1 commit
-
-
Felipe Artur authored
-
- 30 May, 2016 13 commits
-
-
Grzegorz Bizon authored
See #17478
-
Grzegorz Bizon authored
No spaces before commas. See #17478
-
Grzegorz Bizon authored
See #17478
-
Grzegorz Bizon authored
See #17478
-
Grzegorz Bizon authored
Use Array.new with a block instead of `.times.collect` / `.times.map`. See #17478
-
Grzegorz Bizon authored
Favor `unless` over `if` for negative conditions (or control flow ||). See #17478
-
Grzegorz Bizon authored
Checks of literals used in conditions. See #17478
-
Grzegorz Bizon authored
Do not use then for multi-line if. See #7478.
-
Grzegorz Bizon authored
Use def with parentheses when there are parameters. See #17478
-
Grzegorz Bizon authored
See #17478
-
Grzegorz Bizon authored
See #17478
-
Grzegorz Bizon authored
Use Unix-style line endings. See #17478.
-
Grzegorz Bizon authored
See #17478
-
- 25 May, 2016 2 commits
-
-
Robert Speicher authored
[ci skip]
-
Grzegorz Bizon authored
Closes #17406
-
- 24 May, 2016 1 commit
-
-
Robert Speicher authored
Also removes the note from the development/testing.md guide
-
- 23 May, 2016 6 commits
-
-
Robert Speicher authored
[ci skip]
-
Robert Speicher authored
Almost all cops are starting as disabled until we can fix their violations.
-
Grzegorz Bizon authored
This enables multiple Rubocop cops that already conform to our codebase and do not require fixes.
-
Robert Speicher authored
-
Robert Speicher authored
-
Grzegorz Bizon authored
-