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

Simplify Git LFS documentation in the Developer Guide

    XMLWordPrintable

    Details

      Description

      With the Git LFS updates, the Git LFS documentation (https://developer.lsst.io/tools/git_lfs.html) as written seems to be unclear. This ticket will clean that up.

        Attachments

          Activity

          Hide
          jsick Jonathan Sick added a comment -
          • Slim down the installation docs to rely on documentation from Git LFS/GitHub itself.
          • Change the configuration documentation model so that everyone runs through the configuration steps. A second section covers the `git push` authentication pattern.

          J Matt Peterson [X] What permissions are needed by the GitHub personal token? I can include that at the same time.

            1. Pages updated

          https://developer.lsst.io/v/DM-11002/tools/git_lfs.html

          Show
          jsick Jonathan Sick added a comment - Slim down the installation docs to rely on documentation from Git LFS/GitHub itself. Change the configuration documentation model so that everyone runs through the configuration steps. A second section covers the `git push` authentication pattern. J Matt Peterson [X] What permissions are needed by the GitHub personal token? I can include that at the same time. Pages updated https://developer.lsst.io/v/DM-11002/tools/git_lfs.html
          Hide
          jmatt J Matt Peterson [X] (Inactive) added a comment -

          You only need read:org permissions for git-lfs.

          Show
          jmatt J Matt Peterson [X] (Inactive) added a comment - You only need read:org permissions for git-lfs.
          Hide
          jmatt J Matt Peterson [X] (Inactive) added a comment -

          This looks like a great improvement overall.

          It's missing the required permissions when using a personal access token. But otherwise is complete.

          Show
          jmatt J Matt Peterson [X] (Inactive) added a comment - This looks like a great improvement overall. It's missing the required permissions when using a personal access token. But otherwise is complete.
          Hide
          jmatt J Matt Peterson [X] (Inactive) added a comment -

          After looking at the PR it looks like a sentence with permissions for the PAT has been added. Excellent.

          Show
          jmatt J Matt Peterson [X] (Inactive) added a comment - After looking at the PR it looks like a sentence with permissions for the PAT has been added. Excellent.
          Hide
          jsick Jonathan Sick added a comment -
          Show
          jsick Jonathan Sick added a comment - Thanks J Matt Peterson [X]

            People

            Assignee:
            jsick Jonathan Sick
            Reporter:
            jsick Jonathan Sick
            Reviewers:
            J Matt Peterson [X] (Inactive)
            Watchers:
            Frossie Economou, J Matt Peterson [X] (Inactive), Jonathan Sick
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                CI Builds

                No builds found.