Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using SSH_AUTH_SOCK (ssh agent forwarding) to pull upm private repos #124

Conversation

ivan-hernandez-scopely
Copy link
Contributor

@ivan-hernandez-scopely ivan-hernandez-scopely commented May 18, 2021

Changes

Usage


      - name: Setup SSH Agent
        uses: webfactory/[email protected]
        with:
          ssh-private-key: ${{ secrets.SSH_PRIVATE_KEY }}

      - name: Unity Test Runner (custom)
        uses: ivan-hernandez-scopely/unity-test-runner@feature/ssh-agent-for-private-upm
        id: tests
        env:
          UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
          UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
          UNITY_SERIAL: ${{ secrets.UNITY_SERIAL }}
        with:
          projectPath: ${{ matrix.projectPath }}
          testMode: ${{ matrix.testMode }}
          artifactsPath: ${{ matrix.testMode }}-artifacts
          sshAgent: ${{ env.SSH_AUTH_SOCK }}

Checklist

  • Read the contribution guide and accept the code of conduct
  • Readme (updated or not needed)
  • Tests (added, updated or not needed)

@github-actions
Copy link

Cat Gif

@webbertakken
Copy link
Member

Great work once again.

We'll merge this at the same time as game-ci/unity-builder#256, after we've resolved any discussions.

@webbertakken webbertakken merged commit 19661e2 into game-ci:main May 28, 2021
@webbertakken
Copy link
Member

webbertakken commented May 28, 2021

Great work. Thank you very much!

Note that this will start working only after 0.14 of the docker images has been released.

@ivan-hernandez-scopely ivan-hernandez-scopely deleted the feature/ssh-agent-for-private-upm branch May 31, 2021 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants