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

tox indexserver is deprecated; use PIP_INDEX_URL and PIP_EXTRA_INDEX_URL in dev environments #597

Merged
merged 1 commit into from
Mar 31, 2023

Conversation

rrjbca
Copy link
Contributor

@rrjbca rrjbca commented Mar 26, 2023

Description

The tox indexserver configuration has been removed. This is causing the compatibility workflow to fail for dev environments which attempt to install numpy and scipy from the scipy-nightly-wheels server. This pull request modifies the tox dev environments to instead set the PIP_INDEX_URL and PIP_EXTRA_INDEX_URL environment variables to scipy-nightly-wheels and PyPI respectively.

I have tested this on a branch where I modified the compatibility workflow to be run on git pushes (rather than a cron schedule):

(Note that the passing tests installed numpy==1.25.0.dev0+941.gb35aac2c3 and scipy==1.11.0.dev0 develop versions from scipy-nightly-wheels while all other dependencies are releases installed from PyPI as intended).

Checklist

  • Follow the Contributor Guidelines
  • Write unit tests
  • Write documentation strings
  • Assign someone from your working team to review this pull request
  • Assign someone from the infrastructure team to review this pull request

Copy link
Contributor

@itrharrison itrharrison left a comment

Choose a reason for hiding this comment

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

LGTM. And I learned something about the usefulness of compatability tests with the nightly builds.

@itrharrison itrharrison merged commit 915d48f into skypyproject:main Mar 31, 2023
rrjbca added a commit to rrjbca/skypy that referenced this pull request Mar 31, 2023
@rrjbca rrjbca deleted the tox_indexserver branch March 31, 2023 15:30
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.

2 participants