-
Notifications
You must be signed in to change notification settings - Fork 168
Add note to convert ported docstrings #3055
Conversation
|
||
PUP-2 TODO(asmacdo)link adopted the use of Google Style Docstrings TODO(asmacdo)link. When developers port code from Pulp 2 to Pulp 3, they should also convert all the docstrings to the new style. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went to review this, but it looks like it is still a work in progress. We shouldn't commit TODOs to the docs so maybe put a WIP? I can review whenever if you put a comment w/ my nick on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed that change in git. The corrected links are pushed.
ec27fac
to
ffec23e
Compare
@bmbouter ready for review |
|
||
`PUP-2 <https://github.com/pulp/pups/blob/master/pup-0002.md>`_ adopted Google style for :ref:`google-docstrings`. When developers port code from Pulp 2 to Pulp 3, they should also convert all the docstrings to the new style. vim-style regexes can be used to speed up the process. Together, these will convert all of the parameters to Google Style:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra space at the beginning of this line is causing styling issue
|
||
`PUP-2 <https://github.com/pulp/pups/blob/master/pup-0002.md>`_ adopted Google style for :ref:`google-docstrings`. When developers port code from Pulp 2 to Pulp 3, they should also convert all the docstrings to the new style. vim-style regexes can be used to speed up the process. Together, these will convert all of the parameters to Google Style:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/developers port/porting/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/they should//
ffec23e
to
e0dbed7
Compare
https://pulp.plan.io/issues/2347