Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-16860

Convert afw.formatters to numpydoc

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:

      Description

      Convert all Doxygen-formatted docstrings in afw.formatters to Numpydoc style, and move corresponding topic documentation to doc/.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          No builds found.
          krzys Krzysztof Findeisen created issue -
          krzys Krzysztof Findeisen made changes -
          Field Original Value New Value
          Epic Link DM-16722 [ 235355 ]
          krzys Krzysztof Findeisen made changes -
          Assignee Krzysztof Findeisen [ krzys ]
          Status To Do [ 10001 ] In Progress [ 3 ]
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment -

          Hi John Swinbank, could you please review this (very trivial) ticket?

          Show
          krzys Krzysztof Findeisen added a comment - Hi John Swinbank , could you please review this (very trivial) ticket?
          krzys Krzysztof Findeisen made changes -
          Reviewers John Swinbank [ swinbank ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          krzys Krzysztof Findeisen made changes -
          Story Points 0
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Looks fine; thank you.

          You might take this opportunity to remove the docstring from __init__.py since it's useless (and you are explicitly disabling its display). And if you're editing that file, why not take the opportunity to upgrade it to the modern license statement, even though it's technically outside the scope of this ticket?

          Show
          swinbank John Swinbank added a comment - Looks fine; thank you. You might take this opportunity to remove the docstring from __init__.py since it's useless (and you are explicitly disabling its display). And if you're editing that file, why not take the opportunity to upgrade it to the modern license statement, even though it's technically outside the scope of this ticket?
          swinbank John Swinbank made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment -

          Thanks, I hadn't noticed that somebody already took care of afw/COPYRIGHT. I'll add the change.

          Show
          krzys Krzysztof Findeisen added a comment - Thanks, I hadn't noticed that somebody already took care of afw/COPYRIGHT . I'll add the change.
          krzys Krzysztof Findeisen made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          swinbank John Swinbank made changes -
          Story Points 0
          swinbank John Swinbank made changes -
          Story Points 0.1

            People

            Assignee:
            krzys Krzysztof Findeisen
            Reporter:
            krzys Krzysztof Findeisen
            Reviewers:
            John Swinbank
            Watchers:
            John Swinbank, Krzysztof Findeisen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.