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

rubocop -a infinite colons on an empty comment annotation #1901

Closed
l8nite opened this issue May 16, 2015 · 0 comments · Fixed by #1908
Closed

rubocop -a infinite colons on an empty comment annotation #1901

l8nite opened this issue May 16, 2015 · 0 comments · Fixed by #1908

Comments

@l8nite
Copy link

l8nite commented May 16, 2015

Here is a small file that reproduces the issue:

$ cat test.rb
#!/usr/bin/env ruby
# TODO:
puts 'This file will cause rubocop to hang'

Here, we can see that rubocop is correctly identifying the Style/CommentAnnotations violation:

$ rubocop
warning: parser/current is loading parser/ruby21, which recognizes
warning: 2.1.6-compliant syntax, but you are running 2.1.5.
Inspecting 1 file
C

Offenses:

test.rb:2:3: C: Style/CommentAnnotation: Annotation keywords should be all upper case, followed by a colon and a space, then a note describing the problem.
# TODO:
  ^^^^^

1 file inspected, 1 offense detected

When running auto-correct, we get a hang that looks like it's stuck in an infinite loop:

$ rubocop -a
warning: parser/current is loading parser/ruby21, which recognizes
warning: 2.1.6-compliant syntax, but you are running 2.1.5.
Inspecting 1 file

^C
Exiting... Interrupt again to exit immediately.
^C$

In fact, the "fixed" file now has a bunch of colons in it:

$ cat test.rb
#!/usr/bin/env ruby
# TODO:: : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : :
puts 'This file will cause rubocop to hang'
$ rubocop -v
0.31.0
$ ruby -v
ruby 2.1.5p273 (2014-11-13 revision 48405) [x86_64-linux]
@l8nite l8nite changed the title rubocop -a hangs on a comment line rubocop -a infinite colons on an empty comment annotation May 16, 2015
rrosenblum added a commit to rrosenblum/rubocop that referenced this issue May 19, 2015
rrosenblum added a commit to rrosenblum/rubocop that referenced this issue May 19, 2015
rrosenblum added a commit to rrosenblum/rubocop that referenced this issue May 24, 2015
rrosenblum added a commit to rrosenblum/rubocop that referenced this issue May 25, 2015
rrosenblum added a commit to rrosenblum/rubocop that referenced this issue May 26, 2015
bbatsov added a commit that referenced this issue May 29, 2015
[Fix #1901] Do not correct comments that are missing a note
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 a pull request may close this issue.

1 participant