Skip to content

Commit

Permalink
compliance: update generate_report.sh to point to compliance directory (
Browse files Browse the repository at this point in the history
  • Loading branch information
WesleyRosenblum authored Jun 10, 2024
1 parent 90063a7 commit 946483a
Show file tree
Hide file tree
Showing 11 changed files with 13,361 additions and 14 deletions.
14 changes: 0 additions & 14 deletions .github/workflows/ci_compliance.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,14 +34,6 @@ jobs:
aws-s3-region: us-west-2
cdn: https://d3fqnyekunr9xg.cloudfront.net

# The `duvet report` command generates some artifacts (specs folder) that
# interfere with detecting uncommitted files. This step cleans up those
# artifacts. Since the cleanup runs prior to the “Extract RFC spec data”
# phase, this is a safe operation.
- name: Cleanup intermediate artifacts
run: rm -r specs
shell: bash

- name: Extract RFC spec data
working-directory: ./compliance
run: ./initialize_duvet.sh
Expand All @@ -51,12 +43,6 @@ jobs:
run: |
# If this fails you need to run `cd compliance && ./compliance/initialize_duvet.sh`
#
# FIXME: https://github.com/aws/s2n-tls/issues/4219
# We generate and commit the spec files to avoid re-downloading them each time in
# the CI (avoid flaky network calls). However, this currently doesn't work in
# s2n-tls since duvet assumes that the specs folder live in the project's base
# folder.
#
# Use 'git status --porcelain' instead of 'git diff --exit-code' since git diff
# only detects diffs but fails to detect new files. Ignore the s2n-quic dir
# `(:!s2n-quic)` since we explicitly clone the repo as part of this job.
Expand Down
1 change: 1 addition & 0 deletions compliance/generate_report.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ pushd $TOPLEVEL > /dev/null

duvet \
report \
--spec-path 'compliance' \
--spec-pattern 'compliance/specs/**/*.toml' \
--source-pattern '(*=,*#)api/**/*.[ch]' \
--source-pattern '(*=,*#)bin/**/*.[ch]' \
Expand Down
Loading

0 comments on commit 946483a

Please sign in to comment.