Skip to content

feat: sbom and wheelhouse filtering on GitHub release#1260

Open
SMoraisAnsys wants to merge 6 commits intomainfrom
feat/allow-sbom-and-wheelhouse-filtering
Open

feat: sbom and wheelhouse filtering on GitHub release#1260
SMoraisAnsys wants to merge 6 commits intomainfrom
feat/allow-sbom-and-wheelhouse-filtering

Conversation

@SMoraisAnsys
Copy link
Copy Markdown
Contributor

@SMoraisAnsys SMoraisAnsys commented Apr 14, 2026

This PR adds support for filtering distribution files in the dist/wheelhouse and dist/sbom directories when releasing on Github.
The main reason for this change is that the combinatorial associated to the build-wheelhouse action can lead to a releases page heavily loaded, making it hard for users to find the artifact they would like to consume. Below is an example in pyaedt
image
where not all items are showing AND where we have been discarding the wheelhouses without optional target. This is because most (if not all) of our users request the wheelhouse with all target.

While removing the wheelhouses without optional target reduces the number of artifact, it can also lead to a release that is not working without an optional dependency installed :/ This scenario happened in a release of pyaedt where the command import ansys.aedt.core was failing due to a graphic dependency being required. This error went through the CI because we removed the target '' from the build-wheelhouse job as we didn't want to many artifacts in the releases page...

The proposed feature would help to keep having the full build-wheelhouse job and filtering what gets into the releases page.


Here is a successful run where I played with the filter "*ubuntu-latest*,*windows-latest-3.12*,*-all-wheelhouse-macos-latest-3.14*" and which worked as expected.

@github-actions github-actions bot added the enhancement General improvements to existing features label Apr 14, 2026
@SMoraisAnsys SMoraisAnsys changed the title feat: sbom and wheelhouse filtering on gh release feat: sbom and wheelhouse filtering on GitHub release Apr 14, 2026
@SMoraisAnsys SMoraisAnsys marked this pull request as ready for review April 15, 2026 08:04
@SMoraisAnsys SMoraisAnsys requested a review from a team as a code owner April 15, 2026 08:04
Copy link
Copy Markdown
Contributor

@moe-ad moe-ad left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for this @SMoraisAnsys!

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

Labels

enhancement General improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants