Skip to content

Conversation

@Titus-von-Koeller
Copy link
Collaborator

No description provided.

@Titus-von-Koeller Titus-von-Koeller requested a review from Copilot May 15, 2025 14:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the GitHub Actions workflow for the Python package release process. The changes add steps to inspect directories, move and rename wheel files, delete an old pre-release release, generate a release body with pip install commands, and create a new pre-release artifact.

Comment on lines +214 to +217
echo "\`\`\`sh" >> body.md
echo "pip install $url" >> body.md
echo "\`\`\`" >> body.md
echo "" >> body.md
Copy link

Copilot AI May 15, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider refactoring the pip install command generation block by consolidating the multiple echo statements into a single heredoc or a separate script to improve readability and maintainability.

Suggested change
echo "\`\`\`sh" >> body.md
echo "pip install $url" >> body.md
echo "\`\`\`" >> body.md
echo "" >> body.md
cat <<EOF >> body.md
\`\`\`sh
pip install $url
\`\`\`
EOF

Copilot uses AI. Check for mistakes.
uses: softprops/action-gh-release@v2.2.1

- name: Delete old pre-release (if exists)
run: |
Copy link

Copilot AI May 15, 2025

Choose a reason for hiding this comment

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

[nitpick] A brief inline comment explaining the use of '|| true' could help clarify that failing to delete a non-existent release is expected behavior.

Suggested change
run: |
run: |
# Use '|| true' to ensure the script continues even if the release does not exist

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@Titus-von-Koeller Titus-von-Koeller deleted the workflow-patch branch May 15, 2025 14:25
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.

2 participants