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

Clarify travis configuration in dev guide

    XMLWordPrintable

    Details

      Description

      The developer docs about configuring travis are not helpful for understanding what exactly one needs to do to enable it. The docs should state where to put the travis file, how to enable the repo in Travis CI, and should explicitly state how to configure clang-format (not clang-tidy) for a repo, if desired (should we just include that in the default travis configuration?).

      This is the section of interest, particularly starting with part 2. after the first Branch protection image:

      https://developer.lsst.io/stack/adding-a-new-package.html?highlight=travis#configuring-github-repositories

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            The reason I did not document how to enable flake8 and clang-format in travis was that most of our repos would fail if we did that. I'm okay with example .travis.yml files that enable these features (daf_butler has an example flake8 file but I don't think an example clang-format file exists).

            Show
            tjenness Tim Jenness added a comment - The reason I did not document how to enable flake8 and clang-format in travis was that most of our repos would fail if we did that. I'm okay with example .travis.yml files that enable these features (daf_butler has an example flake8 file but I don't think an example clang-format file exists).
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            I recommend using the travis utility to enable/manage travis on a repo. Eg

            gem install -r travis
            travis enable
            

            Show
            jhoblitt Joshua Hoblitt added a comment - I recommend using the travis utility to enable/manage travis on a repo. Eg gem install -r travis travis enable
            Hide
            tjenness Tim Jenness added a comment -

            Closing this because we do not use travis any more.

            Show
            tjenness Tim Jenness added a comment - Closing this because we do not use travis any more.

              People

              Assignee:
              jsick Jonathan Sick
              Reporter:
              Parejkoj John Parejko
              Watchers:
              John Parejko, Jonathan Sick, Joshua Hoblitt, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.