Skip to content

Conversation

@pielas
Copy link
Contributor

@pielas pielas commented Dec 9, 2025

No description provided.

@mkurz
Copy link
Contributor

mkurz commented Dec 9, 2025

Thanks!
Fixes #267

@mkurz
Copy link
Contributor

mkurz commented Dec 9, 2025

@pielas Did you you check against the live api? does it work with packaging=el1,el2,...? Thanks!

@pielas
Copy link
Contributor Author

pielas commented Dec 9, 2025

@mkurz Yes, I tested it with a workflow that failed with previous version and succeeded when using this fix https://github.com/pielas/tapir/actions/runs/20061462893/job/57542386743. By looking at https://api.github.com/repos/pielas/tapir/dependency-graph/snapshots/41383736 I think that github is ignoring this packaging= qualifier and I even thought about removing it completely.

@mkurz
Copy link
Contributor

mkurz commented Dec 9, 2025

@adpi2 Any chance you can release a new version with this fix here? Thanks!

Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

LGTM!

@tgodzik tgodzik merged commit bfbba16 into scalacenter:main Dec 9, 2025
6 checks passed
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.

3 participants