Skip to content

fix(ci): clean up CI warnings and deprecations#91

Merged
JonZeolla merged 6 commits intomainfrom
windows-follow-ons
Apr 5, 2026
Merged

fix(ci): clean up CI warnings and deprecations#91
JonZeolla merged 6 commits intomainfrom
windows-follow-ons

Conversation

@JonZeolla
Copy link
Copy Markdown
Member

Summary

  • Bump GitHub Actions to Node.js 24 compatible versions (actions/cache v5, upload-artifact v7, setup-uv v7, setup-task v2) across root and template workflows
  • Fix coverage "No data was collected" warning by removing pointless --cov from root pytest addopts (the ai_native_python package only has __init__.py metadata)
  • Pass --source-name and --source-version to syft to suppress SBOM warning
  • Redirect stderr on docker buildx inspect to suppress expected ERROR when builder doesn't exist yet
  • Register missing pytest.mark.slow marker in template pyproject.toml
  • Fix argparse prog name in template main.py so --version shows the package name instead of main.py

Test plan

  • CI passes with no Node.js 20 deprecation warnings
  • No "No data was collected" coverage warning in test output
  • No "no explicit name and version" syft warning in SBOM step
  • No ERROR: no builder "multiplatform" found in bootstrap step
  • No PytestUnknownMarkWarning for slow marker in Windows smoke test
  • docker run --rm zenable-io/replace-me:latest --version shows replace_me 0.0.0 instead of main.py 0.0.0

🤖 Generated with Claude Code

- Bump actions/cache v4->v5, upload-artifact v5->v7, setup-uv v4->v7,
  setup-task v1->v2 to resolve Node.js 20 deprecation warnings
- Remove pointless --cov addopts from root pyproject.toml (ai_native_python
  package only has __init__.py metadata, causing "No data collected" warning)
- Add --source-name and --source-version to syft SBOM command to suppress
  "no explicit name and version" warning
- Redirect stderr on docker buildx inspect to suppress expected ERROR when
  multiplatform builder doesn't exist yet
- Register missing pytest.mark.slow marker in template pyproject.toml
- Fix argparse prog name in template main.py (showed "main.py" instead of
  package name in --version output)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! 😎

I didn't find anything of concern

Reviewed with 🤟 by Zenable

JonZeolla and others added 5 commits April 5, 2026 08:40
…verage section

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ess-based tests

Keep --cov and fail_under=80. Add --cov=hooks alongside ai_native_python.
Suppress the "No data was collected" CoverageWarning since tests invoke
hooks via cookiecutter subprocess calls that coverage can't track directly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…r false positives

- Disable uv cache and ignore empty workdir in Windows Smoke Test
  (no checkout step due to NTFS-illegal template dir characters)
- Skip JSON data lines in scan_workflow_logs.sh to avoid false positives
  from SBOM package names like "deprecated"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…oject name

- Change --version output from "prog_name 0.0.0" to just "0.0.0"
- Use "ci-test-project" instead of default "replace-me" for Windows
  smoke test generated project name

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@JonZeolla JonZeolla merged commit 8010670 into main Apr 5, 2026
5 checks passed
@JonZeolla JonZeolla deleted the windows-follow-ons branch April 5, 2026 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant