You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Twine supports the following - TWINE_USERNAME, TWINE_PASSWORD, TWINE_REPOSITORY, TWINE_REPOSITORY_URL, TWINE_CERT, TWINE_NON_INTERACTIVE
These all map to options that are set with Cleo based options in src/slap/ext/application/publish.py that currently get passed through to Twine from slap-cli.
I'd propose simply recrafting all the respective options with something like below as appropriate for each -
Currently the slap-cli publish sub-command does not honor the
TWINE_*
environment variables where-as twine-cli utility does -https://twine.readthedocs.io/en/stable/index.html#configuration
Twine supports the following - TWINE_USERNAME, TWINE_PASSWORD, TWINE_REPOSITORY, TWINE_REPOSITORY_URL, TWINE_CERT, TWINE_NON_INTERACTIVE
These all map to options that are set with Cleo based options in
src/slap/ext/application/publish.py
that currently get passed through to Twine from slap-cli.I'd propose simply recrafting all the respective options with something like below as appropriate for each -
For reference Cleo option.py looks like it should be okay with this.
Seems like a pretty simple thing, but before I charge right in and submit a PR, is this something that sounds about right to you?
The text was updated successfully, but these errors were encountered: