Skip to content

guides: replace pep517 CLI usage with pypa/build#796

Merged
pradyunsg merged 2 commits intopypa:masterfrom
FFY00:replace-pep517-cli
Nov 21, 2020
Merged

guides: replace pep517 CLI usage with pypa/build#796
pradyunsg merged 2 commits intopypa:masterfrom
FFY00:replace-pep517-cli

Conversation

@FFY00
Copy link
Copy Markdown
Member

@FFY00 FFY00 commented Nov 20, 2020

See pypa/pyproject-hooks#91

Signed-off-by: Filipe Laíns lains@riseup.net

@FFY00 FFY00 force-pushed the replace-pep517-cli branch from 9d3b10b to 32fb039 Compare November 20, 2020 17:39
--out-dir dist/
build
--sdist
--weel
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Typo, I assume?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, ugh, sorry. I will fix it up when I get to the computer.

Comment thread source/guides/github-actions-ci-cd-sample/publish-to-test-pypi.yml Outdated
Copy link
Copy Markdown
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

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

LGTM overall

See pypa/pyproject-hooks#91

Signed-off-by: Filipe Laíns <lains@riseup.net>
@FFY00 FFY00 force-pushed the replace-pep517-cli branch from abad790 to 208d121 Compare November 21, 2020 00:28
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
@pradyunsg pradyunsg merged commit 079f4ec into pypa:master Nov 21, 2020
@pradyunsg
Copy link
Copy Markdown
Member

Great! Thanks! ^>^

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.

4 participants