sds / haml-lint

Tool for writing clean and consistent HAML
MIT License
318 stars 101 forks source link

Add autocorrections #524

Open bogdan opened 1 month ago

bogdan commented 1 month ago

Most of haml-lint concerns are too minor to be fixed manually, especially if applied to existing code base. Please introduces the autocorrect at least for the following easy to fix rules:

MaxLap commented 1 month ago

That's a pretty strong case of "do (more) work for me for free". Maybe it's a barrier of language thing, but that phrasing really feels like it.

bogdan commented 1 month ago

Sorry, I didn't mean to offend you. This is how people sometimes frame their requests in my open source repos and I mostly agree because I enjoy helping people by doing things I enjoy to do. I think "I do more work for you for free" is a principle many open source authors follow. At least I do and met many others who do.

MaxLap commented 1 month ago

I'm not offended, I found it a bit funny. Asking for features can be fine. The way of asking can change the tone dramatically.

"please do X" is a polite tone that a boss uses to essentially give an order to an employee.

"at least for" makes it sound like that's the least the person could do. Like it would be unreasonable not to.

Again, maybe just a barrier of language thing. It's not my repo, i just saw the ticket and felt I had to push back a little.