Merge pull request #78 from casperdcl/ci-convenience

minor CI improvements

.github/workflows/ci.yml

@@ -43,14 +43,18 @@ jobs:
     needs: setup
     runs-on: [self-hosted, gpu]
     strategy:
-      fail-fast: false  # TODO: remove
+      fail-fast: false
       matrix:
         model: ${{ fromJson(needs.setup.outputs.models) }}
+    # if nothing to do, skip without failing
+    if: needs.setup.outputs.models != '[]'
     env:
       VERSION: '1.0.0'
       DOCKER_BUILDKIT: 1
       DOCKER_CLI_EXPERIMENTAL: enabled
       COMPOSE_DOCKER_CLI_BUILD: 1
+    permissions:
+      packages: write
     steps:
     - uses: actions/checkout@v3
     - name: Login to GitHub Container Registry
@@ -74,6 +78,7 @@ jobs:
     steps:
     - run: echo success
   teardown:
+    environment: ${{ github.event_name == 'pull_request_target' && !contains('OWNER,MEMBER,COLLABORATOR', github.event.pull_request.author_association) && 'external' || 'internal' }}
     needs: build-test-push
     if: always()
     runs-on: ubuntu-latest