bazel-distribution icon indicating copy to clipboard operation
bazel-distribution copied to clipboard

pass twine publish args in deploy_pip

Open mgosk opened this issue 10 months ago • 1 comments

What is the goal of this PR?

Be able to pass twine args eg --skip-existing , "--non-interactive"

What are the changes implemented in this PR?

Pass twine publish args in deploy_pip function

Live tested on private repository together with other waiting PRs https://github.com/mgosk/bazel-distribution/tree/bzlmod-with-pypi

mgosk avatar Apr 09 '24 20:04 mgosk

hi @mgosk , thank you for the contribution!

This change would be more explicit and aligned with the other rules in bazel_distribution if the arguments supported by twine were each individually configurable in the rule's arguments.

Something like

        "skip_existing": attr.bool(
            default = False,
            doc = "Continue uploading files if one already exists. (Only valid when uploading to PyPI. Other implementations may not support this.)"
        ),

alexjpwalker avatar Jun 13 '24 10:06 alexjpwalker