Skip to content

CI Adds environment for publishing to pypi#28445

Merged
adrinjalali merged 1 commit intoscikit-learn:mainfrom
thomasjpfan:add_environment_to_pypi_upload_v2
Feb 19, 2024
Merged

CI Adds environment for publishing to pypi#28445
adrinjalali merged 1 commit intoscikit-learn:mainfrom
thomasjpfan:add_environment_to_pypi_upload_v2

Conversation

@thomasjpfan
Copy link
Copy Markdown
Member

@thomasjpfan thomasjpfan commented Feb 17, 2024

For publishing to PyPI, the Trusted Publisher docs recommends placing the workflow in an environment. I already created the publish_pypi environment in this repo.

After this is merged and backported to 1.4.X, I'll update the PyPI config to only allow uploads from this environment.

CC @lesteve

For completeness here is what the Trusted Publisher PyPI doc says:

Configuring an environment is optional, but strongly recommended: with a GitHub environment, you can apply additional restrictions to your trusted workflow, such as requiring manual approval on each run by a trusted subset of repository maintainers.

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 04dd3a7. Link to the linter CI: here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants