Adding \n to all Args In The Packages Docstrings. #308
taylor-ennen
started this conversation in
Ideas
Replies: 1 comment 4 replies
-
|
Beta Was this translation helpful? Give feedback.
4 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Before I perform the work or write a script to do so, I'm hoping the maintainers of the project can weigh in on this change.
Overall it is quite simple:
Adding a \n to the end of a Docstring Arg vastly improves readability given different screen and window sizes, bringing a format into place. This would also be implementable if there is a standard format being implemented before changes can be made, ie. Black format.
I'll attach a couple of screen shots below that illustrate this change and the readability I mentioned above.
Without the suggested changes:
With the suggested changes:
@sonichi I believe this is something I can accomplish on my own if you agree to the proposed changes in formatting standards, I can also help to add to the
formatting tool
used if there is one, I have not looked exhaustively though for a standard formatting tool at this point though.Side note: I love the project and am working on implementing a Local Inference with a program LM Studio when I noticed these changes could be made, great work on this whole project; to all the contributors!
Beta Was this translation helpful? Give feedback.
All reactions