Skip to content

CI Adds environment to upload_anaconda#28441

Merged
adrinjalali merged 1 commit intoscikit-learn:mainfrom
thomasjpfan:add_environment_to_upload_wheel_v3
Feb 17, 2024
Merged

CI Adds environment to upload_anaconda#28441
adrinjalali merged 1 commit intoscikit-learn:mainfrom
thomasjpfan:add_environment_to_upload_wheel_v3

Conversation

@thomasjpfan
Copy link
Copy Markdown
Member

Reference Issues/PRs

Related to #24491 (comment)

What does this implement/fix? Explain your changes.

I added a new upload_anaconda environment to the repo: https://github.com/scikit-learn/scikit-learn/settings/environments with the tokens for uploading to anaconda.

With environments, the only GitHub jobs marked with environment: upload_anaconda can have access to the secrets.

CC @lesteve

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

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

Generated for commit: cb7864b. Link to the linter CI: here

Copy link
Copy Markdown
Member

@lesteve lesteve left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for looking into this!

I guess we will be able to remove the identically named repository secrets once this PR is merged?

@lesteve
Copy link
Copy Markdown
Member

lesteve commented Feb 17, 2024

There is probably some clean-up that we can do in the repository secrets already, for example we could remove the PyPI upload token if we are not using it anymore according to #24491 (comment)

@adrinjalali adrinjalali merged commit d6756d7 into scikit-learn:main Feb 17, 2024
@thomasjpfan thomasjpfan added this to the 1.4.2 milestone Feb 17, 2024
@thomasjpfan thomasjpfan added the To backport PR merged in master that need a backport to a release branch defined based on the milestone. label Feb 17, 2024
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build / CI To backport PR merged in master that need a backport to a release branch defined based on the milestone.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants