Skip to content

Commit

Permalink
style-change: update the recommended line wrap from 120 to 190 chars
Browse files Browse the repository at this point in the history
The reason for the change is that this codebase has many functions
with a lot of parameters and inheritances.
Wrapping these functions at 120 characters makes them less readable.
The hit is so noticeable that you really want to make lines longer.
  • Loading branch information
shpaass committed Sep 15, 2024
1 parent 8acc87d commit 61797a8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Docs/CodeStyle.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ The main idea is to keep the code maintainable and readable.

### Code
* In the programmers' haven, there is always a free spot for those who write tests.
* Please try to keep the lines shorter than 120 characters. It simplifies the review of the changes on popular monitors.
* Please try to keep the lines shorter than 190 characters so they fit on most monitors.
* If you add a TODO, then please describe the details in the commit message, or ideally in a github issue. That increases the chances of the TODOs being addressed.

#### Difference with C# conventions
Expand Down

0 comments on commit 61797a8

Please sign in to comment.