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

Licensing in codekit no longer pushes changes.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Somewhere in the codekit refactor we lost the ability to push licensing changes.

        Attachments

          Activity

          No builds found.
          athornton Adam Thornton created issue -
          athornton Adam Thornton made changes -
          Field Original Value New Value
          Epic Link DM-8487 [ 28119 ]
          athornton Adam Thornton made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          Hide
          athornton Adam Thornton added a comment -

          Here's how to test this:

          1) create a repository with a file with an old-style license in it, or just use https://github.com/lsst-sqre/ck-tester.git
          2) create a codekit venv (Python3)
          3) Set up your token file if you need to
          4) lsst-bp -u <your-GH-id> -o lsst-sqre --repo ck-tester --branch fixit
          5) make sure that branch "fixit" of ck-tester has the changes you expect (separate LICENSE/COPYRIGHT, pointer to them in .py file)
          6) delete branch "fixit"
          7) repeat 2-6 with Python 2.

          Show
          athornton Adam Thornton added a comment - Here's how to test this: 1) create a repository with a file with an old-style license in it, or just use https://github.com/lsst-sqre/ck-tester.git 2) create a codekit venv (Python3) 3) Set up your token file if you need to 4) lsst-bp -u <your-GH-id> -o lsst-sqre --repo ck-tester --branch fixit 5) make sure that branch "fixit" of ck-tester has the changes you expect (separate LICENSE/COPYRIGHT, pointer to them in .py file) 6) delete branch "fixit" 7) repeat 2-6 with Python 2.
          Show
          athornton Adam Thornton added a comment - PR is at: https://github.com/lsst-sqre/sqre-codekit/pull/18
          athornton Adam Thornton made changes -
          Status In Progress [ 3 ] In Review [ 10004 ]
          jhoblitt Joshua Hoblitt made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          frossie Frossie Economou made changes -
          Epic Link DM-8487 [ 28119 ] DM-8475 [ 28107 ]
          athornton Adam Thornton made changes -
          Status Reviewed [ 10101 ] In Review [ 10004 ]
          Hide
          athornton Adam Thornton added a comment -

          PR reviewed and merged.

          Show
          athornton Adam Thornton added a comment - PR reviewed and merged.
          athornton Adam Thornton made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          athornton Adam Thornton made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            athornton Adam Thornton
            Reporter:
            athornton Adam Thornton
            Reviewers:
            Angelo Fausti, Frossie Economou, J Matt Peterson [X] (Inactive), Jonathan Sick, Joshua Hoblitt
            Watchers:
            Adam Thornton, Angelo Fausti, Frossie Economou, J Matt Peterson [X] (Inactive), Jonathan Sick, Joshua Hoblitt
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.