Skip to content

Commit

Permalink
fossa: Run separately, only on push (#957)
Browse files Browse the repository at this point in the history
Currently, the FOSSA analysis is set to run as part of CI. Minus the
fact that it's not really part of the build, its reliance on a secret
means that it won't run for any pull requests made from external forks.

Resolve this by running the FOSSA analysis only when we push to a
branch of the Zap repository.
  • Loading branch information
abhinav committed May 25, 2021
1 parent 56304dc commit cfe34dc
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 5 deletions.
17 changes: 17 additions & 0 deletions .github/workflows/fossa.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
name: FOSSA Analysis
on: push

jobs:

build:
runs-on: ubuntu-latest
if: github.repository_owner == 'uber-go'
steps:
- name: Checkout code
uses: actions/checkout@v2

- name: FOSSA analysis
uses: fossas/fossa-action@v1
with:
api-key: ${{ secrets.FOSSA_API_KEY }}

5 changes: 0 additions & 5 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,6 @@ jobs:

- name: Checkout code
uses: actions/checkout@v2

- name: FOSSA analysis
uses: fossas/fossa-action@v1
with:
api-key: ${{ secrets.FOSSA_API_KEY }}

- name: Load cached dependencies
uses: actions/cache@v1
Expand Down

0 comments on commit cfe34dc

Please sign in to comment.