Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
name: CI

on:
merge_group:
push:
branches: [main, master, develop]
pull_request:
Expand Down Expand Up @@ -95,11 +96,11 @@
run: uv python install ${{ matrix.python-version }}

- name: Install workspace dependencies
run: uv sync --all-extras

Check warning on line 99 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUzx&open=AaBp3HIFwtYwlDuGcUzx&pullRequest=58

Check warning on line 99 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUzw&open=AaBp3HIFwtYwlDuGcUzw&pullRequest=58

- name: Run tests with coverage
run: |
uv run pytest packages/cloudflare-auth/tests \

Check warning on line 103 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUzy&open=AaBp3HIFwtYwlDuGcUzy&pullRequest=58
--cov=packages/cloudflare-auth/src/cloudflare_auth \
--cov-report=xml:coverage-cloudflare-auth.xml \
--cov-report=term-missing \
Expand All @@ -108,10 +109,10 @@
-v

- name: Run type checker
run: uv run basedpyright packages/cloudflare-auth/src/

Check warning on line 112 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUzz&open=AaBp3HIFwtYwlDuGcUzz&pullRequest=58

Check warning on line 112 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz0&open=AaBp3HIFwtYwlDuGcUz0&pullRequest=58

- name: Run linter
run: uv run ruff check packages/cloudflare-auth/

Check warning on line 115 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz2&open=AaBp3HIFwtYwlDuGcUz2&pullRequest=58

Check warning on line 115 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz1&open=AaBp3HIFwtYwlDuGcUz1&pullRequest=58

- name: Upload coverage artifact
if: matrix.python-version == '3.12'
Expand Down Expand Up @@ -150,11 +151,11 @@
run: uv python install ${{ matrix.python-version }}

- name: Install workspace dependencies
run: uv sync --all-extras

Check warning on line 154 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz3&open=AaBp3HIFwtYwlDuGcUz3&pullRequest=58

Check warning on line 154 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz4&open=AaBp3HIFwtYwlDuGcUz4&pullRequest=58

- name: Run tests with coverage
run: |
uv run pytest packages/gcs-utilities/tests \

Check warning on line 158 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz5&open=AaBp3HIFwtYwlDuGcUz5&pullRequest=58
--cov=packages/gcs-utilities/src/gcs_utilities \
--cov-report=xml:coverage-gcs-utilities.xml \
--cov-report=term-missing \
Expand All @@ -163,10 +164,10 @@
-v

- name: Run type checker
run: uv run basedpyright packages/gcs-utilities/src/

Check warning on line 167 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz7&open=AaBp3HIFwtYwlDuGcUz7&pullRequest=58

Check warning on line 167 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz6&open=AaBp3HIFwtYwlDuGcUz6&pullRequest=58

- name: Run linter
run: uv run ruff check packages/gcs-utilities/

Check warning on line 170 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz9&open=AaBp3HIFwtYwlDuGcUz9&pullRequest=58

Check warning on line 170 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz8&open=AaBp3HIFwtYwlDuGcUz8&pullRequest=58

- name: Upload coverage artifact
if: matrix.python-version == '3.12'
Expand Down Expand Up @@ -201,11 +202,11 @@
run: uv python install 3.12

- name: Install dependencies
run: uv sync --all-extras

Check warning on line 205 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz-&open=AaBp3HIFwtYwlDuGcUz-&pullRequest=58

Check warning on line 205 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcUz_&open=AaBp3HIFwtYwlDuGcUz_&pullRequest=58

- name: Run Bandit security scan
run: |
uv run bandit -r packages/cloudflare-auth/src/ packages/gcs-utilities/src/ -c pyproject.toml || true

Check warning on line 209 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcU0A&open=AaBp3HIFwtYwlDuGcU0A&pullRequest=58

Check warning on line 209 in .github/workflows/ci.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HIFwtYwlDuGcU0B&open=AaBp3HIFwtYwlDuGcU0B&pullRequest=58

# Upload combined coverage to Codecov
coverage:
Expand Down
8 changes: 7 additions & 1 deletion .github/workflows/pr-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
name: PR Validation

on:
merge_group:
pull_request:
types: [opened, synchronize, reopened]
branches:
Expand All @@ -17,7 +18,12 @@

# Cancel in-progress runs for same PR
concurrency:
group: pr-validation-${{ github.event.pull_request.number }}
# #ASSUME github.event.pull_request.number is null under merge_group;
# fall back to github.ref (unique per merge-group ref) so a merge-group
# build never collides with, or is cancelled by, a PR build's group.
# #VERIFY confirm no cross-cancellation between a PR run and its
# corresponding merge-group run after the first live merge-queue pass.
group: pr-validation-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true

permissions:
Expand Down Expand Up @@ -72,12 +78,12 @@
run: uv python install 3.12

- name: Install dependencies
run: uv sync --all-extras

Check warning on line 81 in .github/workflows/pr-validation.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJgwtYwlDuGcU0D&open=AaBp3HJgwtYwlDuGcU0D&pullRequest=58

Check warning on line 81 in .github/workflows/pr-validation.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJgwtYwlDuGcU0C&open=AaBp3HJgwtYwlDuGcU0C&pullRequest=58

- name: Run vulture
run: |
echo "## Dead Code Report" >> $GITHUB_STEP_SUMMARY
uv run vulture src/ packages/ --min-confidence 90 | tee vulture-report.txt || true

Check warning on line 86 in .github/workflows/pr-validation.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJgwtYwlDuGcU0E&open=AaBp3HJgwtYwlDuGcU0E&pullRequest=58

Check warning on line 86 in .github/workflows/pr-validation.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJgwtYwlDuGcU0F&open=AaBp3HJgwtYwlDuGcU0F&pullRequest=58
if [ -s vulture-report.txt ]; then
echo '```' >> $GITHUB_STEP_SUMMARY
cat vulture-report.txt >> $GITHUB_STEP_SUMMARY
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/reuse.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
name: REUSE Compliance

on:
merge_group:
# No paths filter: "Check REUSE Compliance" is a REQUIRED status check, so it
# must report on every PR. A paths filter would skip the workflow on changes
# that do not match (e.g. root-only files like renovate.json, since "**/*"
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/security-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
name: Security Analysis

"on":
merge_group:
push:
branches: [main, master]
pull_request:
Expand Down Expand Up @@ -51,10 +52,10 @@
run: uv python install 3.12

- name: Install dependencies
run: uv sync --all-extras

Check warning on line 55 in .github/workflows/security-analysis.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJ-wtYwlDuGcU0H&open=AaBp3HJ-wtYwlDuGcU0H&pullRequest=58

Check warning on line 55 in .github/workflows/security-analysis.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJ-wtYwlDuGcU0G&open=AaBp3HJ-wtYwlDuGcU0G&pullRequest=58

- name: Run Bandit security scan
run: uv run bandit -r src/ -c pyproject.toml -f json -o bandit-report.json || true

Check warning on line 58 in .github/workflows/security-analysis.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--no-build" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJ-wtYwlDuGcU0I&open=AaBp3HJ-wtYwlDuGcU0I&pullRequest=58

Check warning on line 58 in .github/workflows/security-analysis.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=ByronWilliamsCPA_python-libs&issues=AaBp3HJ-wtYwlDuGcU0J&open=AaBp3HJ-wtYwlDuGcU0J&pullRequest=58

- name: Run OSV-Scanner dependency scan
uses: google/osv-scanner-action/osv-scanner-action@9a498708959aeaef5ef730655706c5a1df1edbc2 # v2.3.8
Expand Down
Loading