Skip to content
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

bot autoupdating downstream to new url #39

Closed
martinfleis opened this issue Sep 1, 2023 · 4 comments
Closed

bot autoupdating downstream to new url #39

martinfleis opened this issue Sep 1, 2023 · 4 comments

Comments

@martinfleis
Copy link
Member

While discussing the change of the home for nightlies, @bsipocz mentioned that once all core packages are uploading to a new location, there is a way to automatically open PRs downstream, changing the url from the old scipy one to the new scientific-python.

@bsipocz was there an attempt to do that? I haven't noticed it in any of the packages I'm involved with. Given the scipy repository is now outdated, it would be good to roll this out. I understood that all the necessary pieces were ready.

@matthewfeickert
Copy link
Member

matthewfeickert commented Sep 1, 2023

there is a way to automatically open PRs downstream, changing the url from the old scipy one to the new scientific-python.

I may be wrong, but my understanding was that this would be done by people (maybe scripted) but not using a GitHub Actions workflow.

I haven't noticed it in any of the packages I'm involved with.

@martinfleis can you please link all the projects you want updated here that fit the current definitons of "core" (#30 (comment))?

@martinfleis
Copy link
Member Author

my understanding was that this would be done by people

My understanding was that a bot can do this and that @bsipocz knows how to deploy that bot.

but not using a GitHub Actions workflow.

Probably not by GHA. I am not sure. I may be raising the question in the wrong repo...

can you please link all the projects you want updated here that fit the current definitons of "core"

It is not about core packages, it is about the TEST WITH NIGHTLY WHEELS part of the SPEC 4. A lot of non-core downstream packages were already testing against nightly wheels uploaded to pypi.anaconda.org/scipy-wheels-nightly. All those need to be updated. I can update those I maintain but there is a ton of maintainers not aware of the change. The bot should've resolved that.

@matthewfeickert
Copy link
Member

My understanding was that a bot can do this

Okay great.

It is not about core packages, it is about the TEST WITH NIGHTLY WHEELS part of the SPEC 4.
...
All those need to be updated. I can update those I maintain but there is a ton of maintainers not aware of the change. The bot should've resolved that.

Okay. A thing to note is that the example given in SPEC 4 leaves people open to possible security issues (that should get fixed) and the pattern given in

python -m pip install \
--upgrade \
--pre \
--index-url https://pypi.anaconda.org/scientific-python-nightly-wheels/simple \
--extra-index-url https://pypi.org/simple \
matplotlib

(i.e. the index is scientific-python-nightly-wheels and the extra index is PyPI) should be used instead.

@matthewfeickert
Copy link
Member

I'm going to close this as I think all users are now on the correct URL, but please comment if this isn't the case for anyone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants