Skip to content

Conversation

@grische
Copy link
Contributor

@grische grische commented Oct 17, 2025

Split from #47

Line length of 80 vs 88

I removed the line-length of 80 from the pyproject.toml, as it reduced the changes down to 60% (182 vs 117 inserts):

With a line-length of 80:

All done! ✨ 🍰 ✨
5 files reformatted, 2 files left unchanged.
 morgan/__init__.py     |  91 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------
 morgan/configurator.py |  39 ++++++++++++++++++++-------------------
 morgan/metadata.py     |  34 +++++++++++++++++++++-------------
 morgan/server.py       | 115 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------------------------------
 morgan/utils.py        |  13 +++++++------
 pyproject.toml         |   7 +++++++
 6 files changed, 182 insertions(+), 117 deletions(-)

With black's default (=88):

All done! ✨ 🍰 ✨
5 files reformatted, 2 files left unchanged.
 morgan/__init__.py     |  18 ++++++++----------
 morgan/configurator.py |  39 ++++++++++++++++++++-------------------
 morgan/metadata.py     |  34 +++++++++++++++++++---------------
 morgan/server.py       | 113 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------------------
 morgan/utils.py        |  13 +++++++------
 pyproject.toml         |   8 +++++++-
 6 files changed, 117 insertions(+), 108 deletions(-)

Remove line-length setting from pyproject.toml configuration
@ido50 ido50 merged commit c36f2f0 into ido50:main Oct 17, 2025
6 checks passed
@grische grische deleted the patch-1 branch October 17, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants