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

Rework the automation for generating the scripts #85

Merged
merged 8 commits into from
Feb 6, 2021

Conversation

pradyunsg
Copy link
Member

@pradyunsg pradyunsg commented Jan 24, 2021

This is essentially a rewrite of the entire automation in the repository. We're still doing the exact same things, but the plumbing is different now.

  • It's all nox.
    • nox -s generate which handles generating the script.
    • nox -s update-for-release which does everything that a pip release manager might want to do.
    • nox -s check which ensure that the scripts do install pip. :)
  • We got caching now!
    • This means that we're not downloading the same pip version multiple times, which is great!
  • We got "real" CI now.
    • It checks that the generated get-pip.py scripts are the same as what's committed.

The automation is now broken up into smaller composable functions, that each does one this. This should make it easier to tweak how stuff works in the future (like for #61 😉).

This reintroduces the logic we had for generating our get-pip.py scripts
with 3 major changes:

- drop dependency on invoke.
- add caching using requests and cachecontrol.
- setup a nox session to orchestrate everything.
This handles all the git interactions as well, simplifying the process
for whoever makes a pip release.
@pradyunsg
Copy link
Member Author

I'm likely going to be merging this over the coming weekend, since I'd like to get around to cleaning up the error messaging situation we have, and I don't want to touch the templates in this PR. :)

@pradyunsg pradyunsg merged commit 667abf5 into pypa:master Feb 6, 2021
@pradyunsg pradyunsg deleted the rework-how-stuff-works branch February 6, 2021 15:19
@pradyunsg
Copy link
Member Author

@pypa/pip-committers I've changed the automation for this repository here, and the documentation has been updated as well.

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.

1 participant