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

Add rubocop config for each language #2807

Merged
merged 1 commit into from
Nov 30, 2020
Merged

Conversation

feelepxyz
Copy link
Contributor

At some point rubocop removed support for finding configs in parent
directories and only loads the .rubocop.yml defined in the current
folder so we used to run CI lint with -c ../.rubocop.yml but this
breaks editors (e.g. vscode) support as it needs to run rubocop in the
package manager folder.

At some point rubocop removed support for finding configs in parent
directories and only loads the .rubocop.yml defined in the current
folder so we used to run CI lint with `-c ../.rubocop.yml` but this
breaks editors (e.g. vscode) support as it needs to run rubocop in the
package manager folder.
@feelepxyz feelepxyz requested a review from a team as a code owner November 30, 2020 14:51
Copy link
Member

@jurre jurre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this is a great improvement! Not having my editor autocorrect changes has added some unnecessary churn on PRs

@feelepxyz feelepxyz merged commit 4a0e9a8 into main Nov 30, 2020
@feelepxyz feelepxyz deleted the feelepxyz/fix-rubocop-config branch November 30, 2020 15:08
feelepxyz added a commit that referenced this pull request Mar 8, 2021
v7.6.1 (2021-03-04) https://github.com/npm/cli/releases/tag/v7.6.1

BUG FIXES
3c9a589b0 #2807 npm explain show when an edge is a bundled edge (@kumavis)
b33c760ce #2766 unused arguments cleanup (@sandersn)
4a5dd3a5a #2772 fix(npm) pass npm context everywhere (@wraithgar)
e69be2ac5 #2789 fix npm prefix on all Windows unix shells (@isaacs)
2d682e4ca #2803 fix(search): don't pass unused args (@wraithgar)
b3e7dd19b #2822 fix(diff): set option "where" for pacote (@ruyadorno)
96006640b #2824 fix(repo, auth.sso): don't promisify open-url (@wraithgar)

DOCUMENTATION
c8b73db82 #2690 fix(docs): update scripts docs (@wraithgar)
5d922394b #2809 update republish timeout after unpublish (@BAJ-)

DEPENDENCIES
2d4ae598f @npmcli/arborist@2.2.6
feelepxyz added a commit that referenced this pull request Mar 8, 2021
v7.6.1 (2021-03-04) https://github.com/npm/cli/releases/tag/v7.6.1

BUG FIXES
3c9a589b0 #2807 npm explain show when an edge is a bundled edge (@kumavis)
b33c760ce #2766 unused arguments cleanup (@sandersn)
4a5dd3a5a #2772 fix(npm) pass npm context everywhere (@wraithgar)
e69be2ac5 #2789 fix npm prefix on all Windows unix shells (@isaacs)
2d682e4ca #2803 fix(search): don't pass unused args (@wraithgar)
b3e7dd19b #2822 fix(diff): set option "where" for pacote (@ruyadorno)
96006640b #2824 fix(repo, auth.sso): don't promisify open-url (@wraithgar)

DOCUMENTATION
c8b73db82 #2690 fix(docs): update scripts docs (@wraithgar)
5d922394b #2809 update republish timeout after unpublish (@BAJ-)

DEPENDENCIES
2d4ae598f @npmcli/arborist@2.2.6
feelepxyz added a commit that referenced this pull request Mar 8, 2021
v7.6.1 (2021-03-04) https://github.com/npm/cli/releases/tag/v7.6.1

BUG FIXES
3c9a589b0 #2807 npm explain show when an edge is a bundled edge (@kumavis)
b33c760ce #2766 unused arguments cleanup (@sandersn)
4a5dd3a5a #2772 fix(npm) pass npm context everywhere (@wraithgar)
e69be2ac5 #2789 fix npm prefix on all Windows unix shells (@isaacs)
2d682e4ca #2803 fix(search): don't pass unused args (@wraithgar)
b3e7dd19b #2822 fix(diff): set option "where" for pacote (@ruyadorno)
96006640b #2824 fix(repo, auth.sso): don't promisify open-url (@wraithgar)

DOCUMENTATION
c8b73db82 #2690 fix(docs): update scripts docs (@wraithgar)
5d922394b #2809 update republish timeout after unpublish (@BAJ-)

DEPENDENCIES
2d4ae598f @npmcli/arborist@2.2.6
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

2 participants