Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Dec 18, 2019

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?

CHANGELOG.md Outdated Show resolved Hide resolved
@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
CHANGELOG.md Outdated Show resolved Hide resolved
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