Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@matsl
Copy link
Collaborator

@matsl matsl commented Jul 27, 2022

What

Doc strings within 80 char limit for hmouse-tag.el.

Why

Warnings, warnings, warnings...

@matsl matsl requested a review from rswgnu July 27, 2022 09:15
@matsl matsl force-pushed the long_doc_strings_hmouse-tag.el branch from 79c7f80 to 6fc6ad3 Compare August 14, 2022 17:24
@matsl matsl requested a review from rswgnu August 14, 2022 17:25
@matsl
Copy link
Collaborator Author

matsl commented Aug 14, 2022

@rswgnu Had to mix committing changes in the GUI with rebase and resolve some more complicated changes manually.

When there are multi line changes I think it will be easier for me if all lines in the docstrings are changed within one suggested commit. I think that is possible by selecting multiple lines when creating the suggestion. Mark the fist line and use the mouse to drag to the last line you want to change.

Mats Lidell and others added 11 commits September 15, 2022 00:25
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
Co-authored-by: Robert Weiner <[email protected]>
@matsl matsl force-pushed the long_doc_strings_hmouse-tag.el branch from 5ab9bd1 to 8aaccb5 Compare September 14, 2022 22:25
@matsl matsl requested a review from rswgnu September 14, 2022 22:27
@matsl matsl requested a review from rswgnu October 3, 2022 19:41
@matsl matsl merged commit 6ca7e2f into master Oct 5, 2022
@matsl matsl deleted the long_doc_strings_hmouse-tag.el branch October 5, 2022 18:06
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.

3 participants