Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sdist not built #58

Closed
samuelcolvin opened this issue Jul 25, 2022 · 2 comments
Closed

sdist not built #58

samuelcolvin opened this issue Jul 25, 2022 · 2 comments

Comments

@samuelcolvin
Copy link

We've just had a problem on watchfiles where we forgot to include source distributions. samuelcolvin/watchfiles#175

I'm not sure what changed between v0.15 and v0.16, could it have been something to do with maturin/maturin-action?

This issue is just an alert in case others have encountered the same problem, I've fixed this in watchfiles, see samuelcolvin/watchfiles#176.

Feel free to close the issue.
@messense
Copy link
Member

Right, it's the changed behavior in maturin 0.13.0, samuelcolvin/watchfiles#176 is the correct fix.

@samuelcolvin
Copy link
Author

thanks, sorry for noise.

@samuelcolvin samuelcolvin mentioned this issue Jul 25, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@messense @samuelcolvin