Skip to content

Conversation

@dhellmann
Copy link
Member

We were missing docs for both get_build_sdist_dependencies() and build_sdist().

Fixes document the override for creating a source distribution #129

@dhellmann dhellmann force-pushed the 129-document-missing-override-function branch 2 times, most recently from 5d9ca26 to 019317d Compare July 5, 2024 15:43
@rd4398
Copy link
Contributor

rd4398 commented Jul 5, 2024

I don't know how important it is but "The build_sdist() function is responsible forr creating a new source" there is an extra r in forr. (line #340) Rest looks good to me +1:

We were missing docs for both get_build_sdist_dependencies() and build_sdist().

Fixes document the override for creating a source distribution python-wheel-build#129
@dhellmann dhellmann force-pushed the 129-document-missing-override-function branch from 019317d to 6c43f7b Compare July 5, 2024 20:42
@dhellmann
Copy link
Member Author

I don't know how important it is but "The build_sdist() function is responsible forr creating a new source" there is an extra r in forr. (line #340) Rest looks good to me +1:

Good catch, thanks!

@dhellmann dhellmann merged commit 3072efb into python-wheel-build:main Jul 8, 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.

3 participants