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

Update The Rails Style Guide URL #88

Merged
merged 1 commit into from
Jul 14, 2019

Conversation

koic
Copy link
Member

@koic koic commented Jul 13, 2019

Follow up rubocop/rubocop#7120.

This PR updates the Rails Style Guide URL.


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.

Follow up rubocop/rubocop#7120.

This PR updates the Rails Style Guide URL.
@koic koic force-pushed the update_rails_style_guide_url branch from 80a065d to 2f781ca Compare July 13, 2019 22:02
@bbatsov
Copy link
Contributor

bbatsov commented Jul 13, 2019

👍

@koic koic merged commit 12bc6db into rubocop:master Jul 14, 2019
@koic koic deleted the update_rails_style_guide_url branch July 14, 2019 03:14
koic added a commit to koic/rubocop-rails that referenced this pull request Aug 16, 2019
Fixes rubocop#88.

This PR fixes style guide URLs when specifying
`rubocop --display-style-guide` option.

The following is an execution using RuboCop Rails.

```console
% bundle exec rubocop --display-style-guide --only Style/StringLiterals
Inspecting 2 files
C.

Offenses:

Gemfile:3:8: C: Style/StringLiterals: Prefer single-quoted strings when
you don't need string interpolation or special
symbols. (https://rails.rubystyle.guide#consistent-string-literals)
source "https://rubygems.org"
       ^^^^^^^^^^^^^^^^^^^^^^

2 files inspected, 1 offense detected
```

The correct URL is `https://rubystyle.guide#consistent-string-literals`
instead of `https://rails.rubystyle.guide#consistent-string-literals`.
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.

2 participants