Skip to content

Conversation

@Stebalien
Copy link
Contributor

I'm also happy to add a screenshot.

related to #159

@alphapapa
Copy link
Owner

I would like to see a screenshot first, yes. Thanks.

@Stebalien Stebalien force-pushed the steb/document-fill-column branch from 3666687 to 5c7166a Compare May 28, 2023 03:12
@Stebalien Stebalien force-pushed the steb/document-fill-column branch from 5c7166a to 6790ba9 Compare May 28, 2023 03:14
@Stebalien
Copy link
Contributor Author

Done.

(I also won't be offended if you feel that this isn't significant enough to include in the readme)

@alphapapa
Copy link
Owner

Thanks, the screenshot helps.

(I also won't be offended if you feel that this isn't significant enough to include in the readme)

I appreciate your saying that. I do think that this kind of information is useful and should be somewhere, but maybe not in the readme/manual. Maybe it would belong best in a wiki on the repo, or in a secondary documentation file showing how to integrate with other libraries.

If you don't mind, I'll sit on this for a while until we decide where best to put it. Thanks.

@alphapapa alphapapa self-assigned this May 28, 2023
@alphapapa alphapapa added documentation Improvements or additions to documentation help wanted Extra attention is needed labels May 28, 2023
@alphapapa alphapapa added this to the Future milestone May 28, 2023
@Stebalien
Copy link
Contributor Author

Yeah, that was my thought as well. The readme could quickly get cluttered that way.

@alphapapa alphapapa force-pushed the master branch 4 times, most recently from a68abe0 to 9da7b8e Compare September 7, 2023 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants