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
- is triggering
-
DM-4318 Mention self-merge for repos.yaml in documents
- Done
Activity
Field | Original Value | New Value |
---|---|---|
Attachment | Screenshot from 2015-07-24 14-36-56.png [ 27027 ] | |
Attachment | Screenshot from 2015-07-24 14-36-19.png [ 27028 ] | |
Attachment | Screenshot from 2015-07-24 14-36-04.png [ 27029 ] | |
Attachment | Screenshot from 2015-07-24 14-35-50.png [ 27030 ] | |
Attachment | Screenshot from 2015-07-24 14-35-09.png [ 27031 ] |
Resolution | Done [ 10000 ] | |
Status | Proposed [ 10805 ] | Adopted [ 10806 ] |
Status | Adopted [ 10806 ] | Implemented [ 11105 ] |