vhdl-style-guide icon indicating copy to clipboard operation
vhdl-style-guide copied to clipboard

Autofix line length violations

Open imd1 opened this issue 4 years ago • 5 comments

Is your feature request related to a problem? Please describe. Line length violations via length_001 are great but require manual fixes. Other tools for other languages are able to automatically format the code to fit line length rules.

Describe the solution you'd like Provide a fix solution to rule length_001

Describe alternatives you've considered Manual changes, which are difficult to get people to buy into, so the rule always is left as a warning and ignored

Additional context Some tools for other languages are even able to move text to previous lines when the lines are too short. Maybe this is asking too much

imd1 avatar Oct 06 '21 07:10 imd1

Manual changes, which are difficult to get people to buy into, so the rule always is left as a warning and ignored

yeah, people in my company disable the rule.

Some tools for other languages are even able to move text to previous lines when the lines are too short. Maybe this is asking too much

Could you point me to these tools. I would like to understand what they can do.

jeremiah-c-leary avatar Oct 07 '21 00:10 jeremiah-c-leary

Could you point me to these tools. I would like to understand what they can do.

I'm currently on a C++ project where clang-format is used as a formatting tool

imd1 avatar Oct 07 '21 10:10 imd1

this ticket is incorrectly assigned as User Feedback?

imd1 avatar Dec 31 '21 09:12 imd1

I moved this to the Triaged state. We may need more discussion after I look at clang-format.

--Jeremy

jeremiah-c-leary avatar Feb 06 '22 15:02 jeremiah-c-leary

Hey @imd1 ,

I have decided not to implement this issue as I think it will cause more problems for users than just fixing the line length issue manually. I can imagine an infinite amount of variations that people would want and I would have to code those variations. It seems like something I could never get correct.

Any thoughts on my reasoning?

I will plan to close this issue sometime after November 5th if I do not hear from you.

Thanks,

--Jeremy

jeremiah-c-leary avatar Oct 10 '22 00:10 jeremiah-c-leary