Skip to content

Conversation

@HGWright
Copy link
Contributor

@HGWright HGWright commented Jan 2, 2024

🚀 Pull Request

Description

#180 and #181 are failing, I believe because they are reliant on each other. Hopefully puttting the changes in a single PR should fix the CI failure.


@HGWright HGWright marked this pull request as ready for review January 3, 2024 12:53
@HGWright HGWright marked this pull request as draft January 3, 2024 14:50
@HGWright HGWright marked this pull request as ready for review January 11, 2024 15:04
@pp-mo
Copy link
Member

pp-mo commented Jan 12, 2024

I think this won't work if any GHA server produces more than one wheel, since different wheels can produce artifacts of the same name, and the v4 upload now disallows over-writing of an existing named artifact.
See notes here

@HGWright
Copy link
Contributor Author

I think this would still fail if it was attempting to pushing to test-pypi and pypi, so i will try and sort that out before you merge this

@HGWright
Copy link
Contributor Author

I think this would still fail if it was attempting to pushing to test-pypi and pypi, so i will try and sort that out before you merge this

on second thoughts, i'm not so sure that this would fail those steps. @pp-mo I think this is mergable and if it does fail pushing to pypi and test-pypi going forwards at least we know where that is coming from

Copy link
Member

@pp-mo pp-mo left a comment

Choose a reason for hiding this comment

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

There are still some aspects of how the naming works, and what gets uploaded that I don't quite follow - i.e. how does pypa/gh-action-pypi-publish actually know what files to take, and how do the wheels uploaded into /wheelhouse get downloaded into /dist with the sdist.

But it does look reasonable by comparison with what we had before,
so I agree that the easiest approach now is just to merge it + see what happens !

@pp-mo pp-mo merged commit b2c1577 into SciTools:main Jan 12, 2024
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