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

CI validation of lsstsw's repos.yaml

    XMLWordPrintable

    Details

      Description

      Having some sort of automatic "lint check" of the repos.yaml file is desirable due to the length of time required to do a full up test of lsstsw. It should be possible to cobble a sanity checker together that can be run from travis-ci.

        Attachments

          Activity

          Hide
          mjuric Mario Juric added a comment -

          Functionality looks good! I'll leave it to Kian-Tat Lim to rule on whether python 3 is OK at this point.

          Show
          mjuric Mario Juric added a comment - Functionality looks good! I'll leave it to Kian-Tat Lim to rule on whether python 3 is OK at this point.
          Hide
          tjenness Tim Jenness added a comment -

          I thought about commenting on Python 3, but urllib is one of those painful packages that changes between 2.7 and 3. Since this is new code that won't be run by end users I didn't see a problem with it being python3 only. It's not really any different to the ruby code that's included in some of the CI system.

          Show
          tjenness Tim Jenness added a comment - I thought about commenting on Python 3, but urllib is one of those painful packages that changes between 2.7 and 3. Since this is new code that won't be run by end users I didn't see a problem with it being python3 only. It's not really any different to the ruby code that's included in some of the CI system.
          Hide
          ktl Kian-Tat Lim added a comment -

          I generally agree that tools can use whatever languages are appropriate. I'm a little worried about the profusion of customization languages, however. The more languages that someone working on our developer infrastructure has to know, the harder it will be to find such a person or the more training will be required before they can contribute. It would be nice to have a catalog of all developer infrastructure dependencies, in addition to the code dependencies.

          Show
          ktl Kian-Tat Lim added a comment - I generally agree that tools can use whatever languages are appropriate. I'm a little worried about the profusion of customization languages, however. The more languages that someone working on our developer infrastructure has to know, the harder it will be to find such a person or the more training will be required before they can contribute. It would be nice to have a catalog of all developer infrastructure dependencies, in addition to the code dependencies.
          Hide
          ktl Kian-Tat Lim added a comment -

          In terms of a ruling for this particular case, I can accept Python 3 because everyone will eventually have to know it anyway. If it were Haskell or Rust or something else it might not be so easy.

          Show
          ktl Kian-Tat Lim added a comment - In terms of a ruling for this particular case, I can accept Python 3 because everyone will eventually have to know it anyway. If it were Haskell or Rust or something else it might not be so easy.
          Hide
          jhoblitt Joshua Hoblitt added a comment -

          In this particular case, since it was a small one-off, and I wasn't sure if I could get 2.7/future/urllib[2] to be forward compatible with python3, I just did it in python3. Its a small amount of code and I'm happy to rewrite it if necessary.

          I did actually think about writing it in golang to see what the response might be...

          Show
          jhoblitt Joshua Hoblitt added a comment - In this particular case, since it was a small one-off, and I wasn't sure if I could get 2.7/future/urllib [2] to be forward compatible with python3, I just did it in python3. Its a small amount of code and I'm happy to rewrite it if necessary. I did actually think about writing it in golang to see what the response might be...

            People

            Assignee:
            jhoblitt Joshua Hoblitt
            Reporter:
            jhoblitt Joshua Hoblitt
            Reviewers:
            Mario Juric, Tim Jenness
            Watchers:
            Frossie Economou, Joshua Hoblitt, Kian-Tat Lim, Mario Juric, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.