Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-75

self-merge policy for lsstsw/etc/repos.yaml

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      ISSUE

      Description

      I propose that a special-snowflake self-merge policy be adopted for https://github.com/lsst/lsstsw/blob/master/etc/repos.yaml on that basis that:

      • this file is CI configuration data in a simple format; there is no logic for a reviewer to evaluate
      • modifications being merged into this file are in the critical path for many development activities
      • a lint script (DM-3151) is run automatically by travis CI on all pushes & PRs to this repository

      Proposed policy:

      A developer may merge change(s) to the repos.yaml file in the lsstsw repo without formal code review if a github PR has been opened for the change(s) and travis CI has returned a success status for that PR.

      (see attached images of github's CI status flags)

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.