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.

Sign up for GitHub

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

Jump to bottom

operator_spaces: fails with $, #181

Closed
elbrujohalcon opened this issue Nov 28, 2014 · 0 comments
Closed

operator_spaces: fails with $, #181

elbrujohalcon opened this issue Nov 28, 2014 · 0 comments
Assignees
@jfacorro
Labels
bug

Comments

@elbrujohalcon
Copy link
Member

These two lines are reported by elvis:

error1() -> [$,, "where $, represents the character ,"].
error2(<<_:10/binary, ",", _/binary>>) -> "where , is in a string but inside a binary".

Both of them are reported as

  - operator_spaces
    - Missing space after "," on line …
jfacorro added a commit that referenced this issue Mar 18, 2015
@jfacorro
[#181] Fix webhook test.
8f1fb61
jfacorro added a commit that referenced this issue Mar 18, 2015
@jfacorro
[#181] Fixed line_length rule test. Removed unused function. Removed …
592a781 
…warnings from line_length fail test case.
jfacorro added a commit that referenced this issue Mar 19, 2015
@jfacorro
[#181] Modified the search algorithm in elvis_code. Added test case f…
ba61e64 
…or operator_spaces rule.
jfacorro added a commit that referenced this issue Mar 19, 2015
@jfacorro
[#181] Changed https to git. Update version of katana dep.
7209920
@jfacorro jfacorro mentioned this issue Mar 19, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jfacorro jfacorro

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@elbrujohalcon @jfacorro