Merge pull request #32420 from ceddy4395/fix-gha-smells

Fix potential github action smells
dotnet · Apr 29, 2024 · 37e318d · 37e318d
2 parents 0c03008 + d385855
commit 37e318d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/merge-live.yml b/.github/workflows/merge-live.yml
@@ -9,6 +9,10 @@ permissions:
 env:
  DOTNET_DOCS_HEAD: main
  DOTNET_DOCS_BASE: live
+# Prevent merging merging at the same time
+concurrency: 
+ group: ${{github.workflow}}-${{github.ref}}
+ cancel-in-progress: true
 jobs:
  default:
  runs-on: ubuntu-latest

diff --git a/.github/workflows/whats-new.yml b/.github/workflows/whats-new.yml
@@ -23,13 +23,14 @@ jobs:
  permissions:
  contents: write
  pull-requests: write
+ if: ${{ github.repository_owner == 'dotnet' }}
 
  # Steps represent a sequence of tasks that will be executed as part of the job
  steps:
  - uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c
 
  - name: "Print manual run reason"
- if: ${{ github.event_name == 'workflow_dispatch' }}
+ if: github.event_name == 'workflow_dispatch'
  run: |
  echo "Reason: ${{ github.event.inputs.reason }}"