Skip to content

ci: fix published smoke workflow yaml#15

Merged
cafitac merged 1 commit intomainfrom
ci/fix-published-smoke-yaml
Apr 30, 2026
Merged

ci: fix published smoke workflow yaml#15
cafitac merged 1 commit intomainfrom
ci/fix-published-smoke-yaml

Conversation

@cafitac
Copy link
Copy Markdown
Owner

@cafitac cafitac commented Apr 30, 2026

Summary

  • Fixes workflow YAML by replacing an unindented heredoc with a single indented printf command for the pipx wrapper.
  • Keeps the pipx wrapper behavior from PR ci: fix pipx setup for published smoke #14 while allowing GitHub Actions to parse both workflow files again.
  • Updates tests to assert the wrapper intent rather than the heredoc form.

Verification

  • uvx --from pyyaml python ... parsed .github/workflows/publish.yml and .github/workflows/published-install-smoke.yml
  • uv run pytest tests/test_published_install_smoke.py -q -> 11 passed
  • uv run pytest tests/ -q -> 145 passed
  • uv run python scripts/check_release_metadata.py -> OK
  • uv run python scripts/smoke_release_readiness.py -> OK
  • npm pack --dry-run -> OK
  • git diff --check -> OK

Context

@cafitac cafitac merged commit 3dc0fde into main Apr 30, 2026
2 checks passed
@cafitac cafitac deleted the ci/fix-published-smoke-yaml branch April 30, 2026 03:14
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