Skip to content

Conversation

@mfiedorowicz
Copy link
Member

This pull request updates the release workflow for the Python SDK to use the correct package naming convention and simplifies the build process by removing unnecessary filename renaming steps.

Workflow improvements:

  • Changed the PYTHON_PACKAGE_NAME environment variable in .github/workflows/release.yaml from netboxlabs-diode-sdk to netboxlabs_diode_sdk to use underscores instead of hyphens, aligning with Python package naming conventions.

Build process simplification:

  • Removed the step that replaced underscores with hyphens in the build filename, as this is no longer needed due to the updated package name format.

Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
/opt/hostedtoolcache/Python/3.11.14/x64/lib/python3.11/site-packages/netboxlabs/diode/sdk
   client.py4062195%329, 390, 395, 399, 516–518, 523, 528, 533, 538, 548, 552, 556, 578, 599, 601, 675, 706, 714, 769
   exceptions.py44393%69, 82–83
TOTAL4622495% 

Tests Skipped Failures Errors Time
126 0 💤 0 ❌ 0 🔥 1.452s ⏱️

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
/opt/hostedtoolcache/Python/3.10.19/x64/lib/python3.10/site-packages/netboxlabs/diode/sdk
   client.py4062195%329, 390, 395, 399, 516–518, 523, 528, 533, 538, 548, 552, 556, 578, 599, 601, 675, 706, 714, 769
   exceptions.py44393%69, 82–83
TOTAL4622495% 

Tests Skipped Failures Errors Time
126 0 💤 0 ❌ 0 🔥 1.473s ⏱️

@mfiedorowicz mfiedorowicz merged commit 4cb09b8 into develop Nov 20, 2025
6 checks passed
@mfiedorowicz mfiedorowicz deleted the fix-gha-release-workflow-package-name branch November 20, 2025 11:08
@github-actions
Copy link

🎉 This PR is included in version 1.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants