Skip to content

[Fix #166] Ignore db/schema and bin#168

Merged
koic merged 1 commit intorubocop:masterfrom
fidalgo:ignore-db-schema-and-bin-directory
Dec 18, 2019
Merged

[Fix #166] Ignore db/schema and bin#168
koic merged 1 commit intorubocop:masterfrom
fidalgo:ignore-db-schema-and-bin-directory

Conversation

@fidalgo
Copy link
Contributor

@fidalgo fidalgo commented Dec 6, 2019

Since these files are generated by Rails, Rucocop should not
report offenses on them.


Before submitting the PR make sure the following are checked:

  • Wrote good commit messages.
  • Commit message starts with [Fix #issue-number] (if the related issue exists).
  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
  • The PR relates to only one subject with a clear title
    and description in grammatically correct, complete sentences.
  • Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

@koic
Copy link
Member

koic commented Dec 6, 2019

@fidalgo fidalgo force-pushed the ignore-db-schema-and-bin-directory branch 2 times, most recently from a46206b to da112b9 Compare December 6, 2019 19:21
@fidalgo
Copy link
Contributor Author

fidalgo commented Dec 8, 2019

@koic added the changelog... I don't know why the Ruby HEAD is failing, do you have any clue?

@koic
Copy link
Member

koic commented Dec 8, 2019

CI failure for Ruby HEAD has been resolved by #169. Can you rebase with the latest master branch?

@fidalgo fidalgo force-pushed the ignore-db-schema-and-bin-directory branch from da112b9 to ee3c159 Compare December 11, 2019 17:08
Since these files are generated by Rails, Rucocop should not
report offenses on them.

Fixes rubocop#166
@fidalgo fidalgo force-pushed the ignore-db-schema-and-bin-directory branch from ee3c159 to b1756fe Compare December 13, 2019 11:00
@koic koic merged commit cbc0673 into rubocop:master Dec 18, 2019
@pirj
Copy link
Member

pirj commented Nov 25, 2020

As AllCops suggests, you're excluding those files for all cops. Was that the intention? Maybe it would have been better under Rails/Exclude?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments