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

Migrate pipe_analysis repo to GitHub Actions

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      DRP S21a (Dec Jan)
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Due to the instability of travis job checks for git repositories, convert the pipe_analysis repository to GitHub Actions (which has proven more reliable).  Follow instructions provided here.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          When you get a chance, would you mind giving this a look to make sure I did things correctly for the migration?  The PR says it has passed 2 successful lint checks, but is hung up on the 2 travis checks.  Will I be able to merge this as I would  do from the command line for a normal ticket?  I.e. 

          git merge --no-ff tickets/DM-NNNN
          		 
          git push

          (the guide just says "9. Create a pull request and merge it.")

          Show
          lauren Lauren MacArthur added a comment - When you get a chance, would you mind giving this a look to make sure I did things correctly for the migration?  The PR  says it has passed 2 successful lint checks, but is hung up on the 2 travis checks.  Will I be able to merge this as I would  do from the command line for a normal ticket?  I.e.  git merge - - no - ff tickets / DM - NNNN git push (the guide just says "9. Create a pull request and merge it.")
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          Sorry, the instructions were assuming that Travis would work reasonably. You can go on to the next steps to disable the Travis checks and enable the lint check; that should allow you to merge normally.

          Show
          ktl Kian-Tat Lim added a comment - Sorry, the instructions were assuming that Travis would work reasonably. You can go on to the next steps to disable the Travis checks and enable the lint check; that should allow you to merge normally.
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          Everything looks good.

          Show
          ktl Kian-Tat Lim added a comment - Everything looks good.
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          Thanks so much Kian-Tat Lim.  Merged to master and all looks good.

          Show
          lauren Lauren MacArthur added a comment - Thanks so much Kian-Tat Lim .  Merged to master and all looks good.

            People

            Assignee:
            lauren Lauren MacArthur
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Kian-Tat Lim
            Watchers:
            Kian-Tat Lim, Lauren MacArthur
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.