Skip to content

CI Adds environment to upload_anaconda #28441

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

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