Skip to content

Conversation

thomasjpfan
Copy link
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

Copy link

✔️ 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
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
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