Skip to content

fix: publish_python_libraries opt-in default + README cleanup#50

Merged
tehw0lf merged 1 commit intomainfrom
feature/npm-publish-provenance
Mar 15, 2026
Merged

fix: publish_python_libraries opt-in default + README cleanup#50
tehw0lf merged 1 commit intomainfrom
feature/npm-publish-provenance

Conversation

@tehw0lf
Copy link
Owner

@tehw0lf tehw0lf commented Mar 15, 2026

Summary

  • publish_python_libraries default changed to "false" — explicit opt-in required (only one Python repo uses this)
  • Removed stale event_name input from all README examples (no longer a workflow input)
  • Added publish_python_libraries: "true" to Python example so it actually works as shown
  • Removed release_tag from Rust example (version is auto-read from Cargo.toml)
  • Fixed duplicate section numbering (two "### 7." entries)

Test plan

  • Verify Python repos that don't publish to PyPI no longer trigger publish_python_libraries job
  • Verify the Python repo that does publish still works by explicitly setting publish_python_libraries: "true"

…flag

- Remove event_name input from all examples (no longer a workflow input)
- Add publish_python_libraries: "true" to Python example (required opt-in)
- Remove release_tag from Rust example (auto-read from Cargo.toml)
- Fix duplicate section numbering (7. appeared twice)
- Default publish_python_libraries to "false" (explicit opt-in required)
@tehw0lf tehw0lf merged commit c9671f4 into main Mar 15, 2026
2 checks passed
@tehw0lf tehw0lf deleted the feature/npm-publish-provenance branch March 15, 2026 16:42
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