Shane da Silva

Results 170 comments of Shane da Silva

Supportive of reducing the friction to integration a separate executable/script with Overcommits line handling functionality. Pull request with tests welcome!

I could see many built-in hooks converted to the ad hoc format, yes. I haven't considered the implications too deeply, however, but would be open to a proposal. Perhaps we...

Thanks for the report. Can you clarify which hook was demonstrating this behavior? Furthermore, can you confirm the version of `git` and `overcommit` you are currently using? (since it's been...

Thanks for your reply. Apologies for the delay in response, as I am traveling. If I simplify what you're proposing down to to its core elements, it appears you are...

@alfonsojimenez implemented this long ago (thank you!) Would love to see a pull request resolving the functional discrepancy—but otherwise agree a documentation change would help clarify.

Open to pull requests addressing this.

Totally agree that this would be awesome to have. Will get to it when I can, otherwise a pull request would be most welcome!

Thanks for the reports, @printercu. I can't reproduce the issue for the first example you give: ```slim - 2.times do |x| div attr=x ``` This doesn't output a warning for...

Hey @jaimerson, This is doable with Slim-Lint, but I think you will find edge cases where static text is indeed what you want, which will lead you to do something...

@mmzoo you added this linter in #82. Can you investigate?