You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, maximum line length is not set in /backend/.editorconfig
If the developer accidentally has triggered autoformatting in their IDE, this can trigger unintended line breaks.
Issues with this:
Can waste time if the developer has to revert the unintended line breaks
Can make existing code harder to read
Can make the diff on GitHub hard to read
Suggestions:
Set max_line_length = off. The developers can then decide the line width themself, without worrying about autoformatting
Set max_line_lenght = 200 This will enforce a collective maximum linewidth, while still leaving ample room for the developer regarding where to put the line breaks
Potential cons:
If the developer has their preferred line length set in the IDE, it will be overrun by the .editorconfig file
The text was updated successfully, but these errors were encountered:
ErlingHauan
added
status/draft
Status: When you create an issue before you have enough info to properly describe the issue.
kind/chore
kind/analysis
Used when an issue needs to be analysed before it becomes a user story.
status/for-consideration
Issues that need consideration by team
and removed
kind/analysis
Used when an issue needs to be analysed before it becomes a user story.
status/draft
Status: When you create an issue before you have enough info to properly describe the issue.
labels
May 14, 2024
ErlingHauan
changed the title
Add max line length to .editorconfig
Consider adding max line length to .editorconfig
May 28, 2024
Description
Currently, maximum line length is not set in
/backend/.editorconfig
If the developer accidentally has triggered autoformatting in their IDE, this can trigger unintended line breaks.
Issues with this:
Suggestions:
max_line_length = off
. The developers can then decide the line width themself, without worrying about autoformattingmax_line_lenght = 200
This will enforce a collective maximum linewidth, while still leaving ample room for the developer regarding where to put the line breaksPotential cons:
The text was updated successfully, but these errors were encountered: