search

hmil / tslint-override

TSLint plugin bringing support for the override keyword
MIT License
27 stars 6 forks source link

Adds fixer-break-line-after option #20

Closed mzyil closed 5 years ago

mzyil commented 5 years ago

Add an option to enforce a new line after jsdoc or decorator tag #19

mzyil commented 5 years ago

@hmil ping

hmil commented 5 years ago

Sweet. That's a really clean patch, I appreciate it. Could you either address or remove the TODO?

hmil commented 5 years ago

I personally find OPTION_NEW_LINE_AFTER_DECORATORS_AND_TAGS a bit too verbose, but whatever. Whichever you chose, just stick to it for all constant and variable names so it's consistent.

mzyil commented 5 years ago

@hmil @thehcma I think all reviews are implemented

hmil commented 5 years ago

Cool. Thanks for your contrib @mzyil . It should be in v0.5.0 if the Travis gods are with us.