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

pronto-rubocop 0.4.4 ignoring rubocop:disable #13

Closed
dankohn opened this issue Jun 17, 2015 · 6 comments
Closed

pronto-rubocop 0.4.4 ignoring rubocop:disable #13

dankohn opened this issue Jun 17, 2015 · 6 comments
Assignees
Labels

Comments

@dankohn
Copy link
Contributor

dankohn commented Jun 17, 2015

pronto rubocop 0.4.4 is ignoring rubocop:disable entries, while 0.4.3 honors them correctly.

Running rubocop on a branch produces a clean result, but running pronto run -r=rubocop shows me 5 Rubocop errors. Each of the Rubocop errors has been explicitly disabled in the code.

Running pronto run -r=rubocop --commit=$(git log --pretty=format:%H | tail -1) shows me dozens of Rubocop errors across the whole codebase.

My repo is proprietary, but I doubt the bug is limited to me. It seems like something changed in how rubocop accepts how it is being evoked by pronto-rubocop, causing it to ignore disables. Please let me know if there's anything else I can report.

@mmozuras mmozuras self-assigned this Jun 18, 2015
@mmozuras mmozuras added the bug label Jun 18, 2015
@ivan-leschinsky
Copy link

@mmozuras, how to fix this?
could you fix it?

@dankohn
Copy link
Contributor Author

dankohn commented Sep 5, 2015

@mmozuras Welcome back from vacation. Any ideas on a fix for this?

@ivan-leschinsky
Copy link

It is problem of rubocop, not pronto-rubocop. It is ignored only when run
without command-line, for example with pronto-rubocop.

сб, 5 сент. 2015, 22:02, Dan Kohn [email protected]:

@mmozuras https://github.com/mmozuras Welcome back from vacation. Any
ideas on a fix for this?


Reply to this email directly or view it on GitHub
#13 (comment)
.

@mmozuras
Copy link
Member

mmozuras commented Sep 5, 2015

@dankohn I'll make this bug a priority. Expect more news tomorrow 😄

@dankohn
Copy link
Contributor Author

dankohn commented Sep 5, 2015

@mmozuras Hugely appreciated. I love pronto. Here I am signing it's praises: https://news.ycombinator.com/item?id=10055342

@mmozuras
Copy link
Member

mmozuras commented Sep 6, 2015

@dankohn so that's where those stars came from... But honestly, can't express how great that feels 😄

And the bug is fixed and released, btw 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants