search

lpil / dogma

:closed_lock_with_key: A code style linter for Elixir
Other
471 stars 52 forks source link

Clean up FunctionParentheses rule #211

Closed mgartner closed 8 years ago

mgartner commented 8 years ago

I realized I could refactor my original code here by simplifying it into two functions that match when there is a rule being broken. I think this code is easier to understand.

Any feedback on how to format these functions with headers that exceed the 80 character line length limit? Let me know what you prefer.

lpil commented 8 years ago

This looks great :)

The way you've formatted the function arguments seems clear and sensible to me,