-
Notifications
You must be signed in to change notification settings - Fork 809
Description
What problem do you want to solve?
Hi there,
I mentioned this in the last meeting: I would like to propose an increase of line length in this repository because the code gets too polluted (not sure if it's the right word, but multiliny?).
Currently, it is set to 79 by pylint:
opentelemetry-python-contrib/.pylintrc
Lines 269 to 270 in 447aac2
| # Maximum number of characters on a single line. | |
| max-line-length=79 |
Describe the solution you'd like
Historically, in Python most projects have been using 88, till some years ago where people started using 120 (my guess is that the type system got too verbosed, and 120 is more acceptable).
In any case, I would like to propose we change it to 120.
The pros are: code far more readable.
The cons are: every PR will need a rebase.
We can add the commit to the .git-revs-file to not lose the history.
Describe alternatives you've considered
No response
Additional Context
No response
Would you like to implement a fix?
Yes
Tip
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.