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

Change the suggested format token style #913

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pirj
Copy link
Member

@pirj pirj commented Dec 30, 2022

According to research:

annotated (%<name>s):
57497 files inspected, 5785 offenses detected

template (%{name}):
57497 files inspected, 1279 offenses detected

unannotated (%s)
57497 files inspected, 6424 offenses detected

the %{name} is the most commonly used style.

Related:

According to [research](rubocop/rubocop#8827 (comment)):

annotated (`%<name>s`):
    57497 files inspected, 5785 offenses detected

template (`%{name}`):
    57497 files inspected, 1279 offenses detected

unannotated (`%s`)
    57497 files inspected, 6424 offenses detected

the `%{name}` is the most commonly used style.
@pirj pirj self-assigned this Dec 30, 2022
@pirj pirj requested a review from a team December 30, 2022 14:21
@pirj pirj requested review from andyw8, bbatsov and koic April 18, 2023 20:17
@andyw8 andyw8 removed their request for review April 24, 2023 13:13
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.

None yet

1 participant