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

Add support for direct URL fetching. #21

Merged
merged 1 commit into from
Dec 5, 2022

Conversation

jsirois
Copy link
Collaborator

@jsirois jsirois commented Dec 5, 2022

The scie-pants project will leverage this in an install binding that
needs to fetch Pants version information from PyPI and GitHub in certain
circumstances.

Along the way, improve CLI help and switch to anyhow for better error
handling.

The scie-pants project will leverage this in an install binding that
needs to fetch Pants version information from PyPI and GitHub in certain
circumstances.

Along the way, improve CLI help and switch to anyhow for better error
handling.
@jsirois jsirois requested review from benjyw and sureshjoshi December 5, 2022 02:31
@jsirois
Copy link
Collaborator Author

jsirois commented Dec 5, 2022

Another ~FYI review. This came out of getting scie-pants working and handling the ./pants PANTS_SHA case as well as handling the one_step_setup.sh case of grabing the latest stable Pants version when configuring a new setup.

@jsirois
Copy link
Collaborator Author

jsirois commented Dec 5, 2022

@chrisjrn I've extended an invite to the repo so I can add you to reviews. This is mainly FYI for the upcoming scie-pants PRs which I'd definitely like your feedback and hawk-eye on.

@chrisjrn
Copy link

chrisjrn commented Dec 5, 2022 via email

@jsirois jsirois merged commit ad12cf6 into a-scie:main Dec 5, 2022
@jsirois jsirois deleted the fetch/support-direct-URL-use branch December 5, 2022 23:35
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

Successfully merging this pull request may close these issues.

2 participants