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

Fix ts_xml doc warnings

    XMLWordPrintable

    Details

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

      Description

      Fix ts_xml doc build warnings.

        Attachments

          Activity

          Show
          ecoughlin Eric Coughlin added a comment - PR  https://github.com/lsst-ts/ts_xml/pull/310
          Hide
          rowen Russell Owen added a comment -

          Can you please point me to a build (showing the warnings gone) and to the built docs?

          I took a preliminary pass on github. It looks fine but I'm not clear on what you changed that fixed the comments. I would appreciate some explanation.

          Show
          rowen Russell Owen added a comment - Can you please point me to a build (showing the warnings gone) and to the built docs? I took a preliminary pass on github. It looks fine but I'm not clear on what you changed that fixed the comments. I would appreciate some explanation.
          Hide
          ecoughlin Eric Coughlin added a comment -

          https://travis-ci.com/github/lsst-ts/ts_xml/builds/174094160 Current develop from 4 days ago, 1695 warnings.

          https://travis-ci.com/github/lsst-ts/ts_xml/builds/174408249 Current PR build 0 warnings.

          • Added a blank line at the end of field lists
          • Added labels in script instead of using the sphinx autolabel extension as they were not unique enough.
          Show
          ecoughlin Eric Coughlin added a comment - https://travis-ci.com/github/lsst-ts/ts_xml/builds/174094160  Current develop from 4 days ago, 1695 warnings. https://travis-ci.com/github/lsst-ts/ts_xml/builds/174408249  Current PR build 0 warnings. Added a blank line at the end of field lists Added labels in script instead of using the sphinx autolabel extension as they were not unique enough.
          Hide
          rowen Russell Owen added a comment -

          Looks good. I had a few suggestions on github (primarily getting rid of <> in section titles and including fixing a few pre-existing issues). If you don't want to ix the pre-existing issues on this ticket please let me know and I'll file a new one.

          Show
          rowen Russell Owen added a comment - Looks good. I had a few suggestions on github (primarily getting rid of <> in section titles and including fixing a few pre-existing issues). If you don't want to ix the pre-existing issues on this ticket please let me know and I'll file a new one.

            People

            Assignee:
            ecoughlin Eric Coughlin
            Reporter:
            ecoughlin Eric Coughlin
            Reviewers:
            Russell Owen
            Watchers:
            Andy Clements, Eric Coughlin, Michael Reuter, Rob Bovill, Russell Owen
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.