Skip to content

Fix for RUBOCOP_CONFIG environment variable#72

Merged
ashkulz merged 1 commit intoprontolabs:masterfrom
haseebab:rubocop_config_env
Dec 31, 2022
Merged

Fix for RUBOCOP_CONFIG environment variable#72
ashkulz merged 1 commit intoprontolabs:masterfrom
haseebab:rubocop_config_env

Conversation

@haseebab
Copy link
Copy Markdown
Contributor

This PR is a proposal to fix #67

It looks like the use of RUBOCOP_CONFIG was removed in this commit: https://github.com/prontolabs/pronto-rubocop/pull/44/files#diff-f8a1b8209086b10d50565dc2776339d453c4bd8783d3c1dfb28907fcad85a125L37

However, the README still mentions that RUBOCOP_CONFIG is supported.

@haseebab haseebab marked this pull request as ready for review December 14, 2021 16:29
@haseebab haseebab requested a review from a team as a code owner December 14, 2021 16:29
@keithrbennett
Copy link
Copy Markdown

keithrbennett commented Dec 30, 2022

@ashkulz , please merge this. It's been over a year since it was proposed, and it looks like a trivial fix. Or if there is any potential problem with it, could you say so? I just wasted a lot of time trusting the documentation. If you don't want to make this fix, then could you at least remove the misleading text from the readme?

Copy link
Copy Markdown
Member

@ashkulz ashkulz left a comment

Choose a reason for hiding this comment

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

Sorry, this slipped through the cracks! It looks like it was mistakenly removed as part of a merge commit in #44 🤷‍♂️

@ashkulz ashkulz merged commit e73a19c into prontolabs:master Dec 31, 2022
@ashkulz
Copy link
Copy Markdown
Member

ashkulz commented Dec 31, 2022

Thanks for the contribution, @haseebab!

@ashkulz
Copy link
Copy Markdown
Member

ashkulz commented Dec 31, 2022

pronto-rubocop 0.11.4 has been released to RubyGems 🎉

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.

Since 0.11.0 RUBOCOP_CONFIG is no longer supported

3 participants