Skip to content

Commit

Permalink
Update FOSSA configuration for new requirements layout (apache#10848)
Browse files Browse the repository at this point in the history
* Update FOSSA configuration

* test FOSSA failure

* Update FOSSA files changed regex

* Revert "test FOSSA failure"

This reverts commit a97213d.

* pre-commit linting

* remove docker.txt check
  • Loading branch information
robdiciuccio authored and auxten committed Nov 20, 2020
1 parent 974fc81 commit d77a36f
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 8 deletions.
8 changes: 2 additions & 6 deletions .fossa.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,9 @@ analyze:
type: npm
target: superset-frontend
path: superset-frontend
- name: docs
type: pip
target: docs
path: docs
- name: .
- name: base
type: pip
target: .
path: .
options:
requirements: ./requirements/testing.txt
requirements: ./requirements/base.txt
2 changes: 1 addition & 1 deletion .github/workflows/license-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
EOF
if [[ "${FILES}" =~ (.*package*\.json|requirements/*\.txt|setup\.py) ]]; then
if [[ "${FILES}" =~ (.*package*\.json|requirements\/[a-z_-]+\.txt|setup\.py) ]]; then
echo "Detected dependency changes... running fossa check"
./scripts/fossa.sh
Expand Down
4 changes: 3 additions & 1 deletion scripts/fossa.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@
#

# This is the recommended way to install FOSSA's cli per the docs:
# https://docs.fossa.com/docs/travisci#section-add-fossa-steps-to-travisyml
# https://docs.fossa.com/docs/generic-ci
curl -s -H 'Cache-Control: no-cache' https://raw.githubusercontent.com/fossas/fossa-cli/master/install.sh | sudo bash

# This key is a push-only API key, also recommended for public projects
# https://docs.fossa.com/docs/api-reference#section-push-only-api-token
export FOSSA_API_KEY="${FOSSA_API_KEY:-f72e93645bdfeab94bd227c7bbdda4ef}"
fossa init
fossa analyze
fossa test

0 comments on commit d77a36f

Please sign in to comment.