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

Help turn on Travis for remaining repositories

    XMLWordPrintable

    Details

    • Type: Epic
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Science Pipelines
    • Labels:
      None
    • Epic Name:
      travisify_F18
    • Story Points:
      20
    • Team:
      External
    • Cycle:
      Fall 2018

      Description

      Most repositories have had the configuration added to turn on flake8 checking on push. There are still a few that need this added. This epic is to help add that configuration and fix any linting issues that cause travis to fail.

        Attachments

          Issue Links

            Activity

            No builds found.
            krughoff Simon Krughoff created issue -
            krughoff Simon Krughoff made changes -
            Field Original Value New Value
            Risk Score 0
            krughoff Simon Krughoff made changes -
            Epic Child DM-15516 [ 181668 ]
            Hide
            krughoff Simon Krughoff added a comment -

            Here is the recipe provided by Russell:

            To enable Travis flake8:

            1: Visit the following page (which eventually will have complete instructions)
            https://developer.lsst.io/stack/adding-a-new-package.html?highlight=travis#configuring-github-repositories

            2: Save the following to the package root as a file named .travis.yaml
            sudo: false
            language: python
            matrix:
            include:

            • python: '3.6'
              install:
            • pip install flake8
              script: flake8

            3: Enable Travis for the repository by visiting:
            https://travis-ci.org/lsst/<repo_name> (e.g. https://travis-ci.org/lsst/afw)
            Click the "Activate" button
            We will change to travis-ci.com at some point, but LSST will say when;
            see https://docs.travis-ci.com/user/open-source-on-travis-ci-com/ for more info

            4: Make sure Travis runs
            If you have a ticket branch then push a change to it. If you are done working on that branch
            then use "git commit --amend" followed by "git push --force" to push a new SHA for your work.
            That should start Travis running. You can see this on github by viewing your branch
            (a pull request may be needed, but I don't think so).

            5: In github (after Travis runs for the first time) you can require it to pass before merging.
            Settings > Branches:
            x Require branches to be up to date before merging (this will already be checked)
            x Continuous Integration Travis/CI

            Show
            krughoff Simon Krughoff added a comment - Here is the recipe provided by Russell: To enable Travis flake8: 1: Visit the following page (which eventually will have complete instructions) https://developer.lsst.io/stack/adding-a-new-package.html?highlight=travis#configuring-github-repositories 2: Save the following to the package root as a file named .travis.yaml sudo: false language: python matrix: include: python: '3.6' install: pip install flake8 script: flake8 3: Enable Travis for the repository by visiting: https://travis-ci.org/lsst/ <repo_name> (e.g. https://travis-ci.org/lsst/afw ) Click the "Activate" button We will change to travis-ci.com at some point, but LSST will say when; see https://docs.travis-ci.com/user/open-source-on-travis-ci-com/ for more info 4: Make sure Travis runs If you have a ticket branch then push a change to it. If you are done working on that branch then use "git commit --amend" followed by "git push --force" to push a new SHA for your work. That should start Travis running. You can see this on github by viewing your branch (a pull request may be needed, but I don't think so). 5: In github (after Travis runs for the first time) you can require it to pass before merging. Settings > Branches: x Require branches to be up to date before merging (this will already be checked) x Continuous Integration Travis/CI
            Hide
            tjenness Tim Jenness added a comment -

            Also look at tests/SConscript and add pyList=[] as an argument to the tests() method there. This enables automatic file detection in pytest when run from sconsUtils.

            Show
            tjenness Tim Jenness added a comment - Also look at tests/SConscript and add pyList=[] as an argument to the tests() method there. This enables automatic file detection in pytest when run from sconsUtils.
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-15589 [ 186614 ]
            krughoff Simon Krughoff made changes -
            Reviewers Simon Krughoff [ krughoff ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            krughoff Simon Krughoff made changes -
            Status In Review [ 10004 ] In Progress [ 3 ]
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-16422 [ 234228 ]
            swinbank John Swinbank made changes -
            Team SQuaRE [ 10302 ] External [ 12117 ]
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-16445 [ 234456 ]
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-16491 [ 234616 ]
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-16493 [ 234622 ]
            jcarlin Jeffrey Carlin made changes -
            Epic Child DM-16520 [ 234720 ]
            frossie Frossie Economou made changes -
            Status Admin Review [ 3 ] In Progress [ 11605 ]
            frossie Frossie Economou made changes -
            Status Review [ 11605 ] In Progress [ 3 ]
            tjenness Tim Jenness made changes -
            Remote Link This issue links to "Page (Confluence)" [ 34367 ]
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status In Progress [ 3 ] Done [ 10002 ]
            Hide
            tjenness Tim Jenness added a comment -

            Closing this as we have long since moved to GitHub actions and enabled the flake8 tests using that for all packages.

            Show
            tjenness Tim Jenness added a comment - Closing this as we have long since moved to GitHub actions and enabled the flake8 tests using that for all packages.

              People

              Assignee:
              jcarlin Jeffrey Carlin
              Reporter:
              krughoff Simon Krughoff
              Reviewers:
              Simon Krughoff
              Watchers:
              Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.