-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Allows usage of UV, CPython and pypy mirrors #147
Allows usage of UV, CPython and pypy mirrors #147
Conversation
5f5a2c4
to
b0802fa
Compare
I added our company to the Users page of the doc 😄 |
@ofek Hi! Thanks for approving the build ! It seems the variables are not in the good order in the validate step:
Any idea how to fix it? 🤔 |
Thanks for this!
|
Oh, makes sense! I will fix it, thanks. 😆
You mean, adding the same fork/mirror configuration by environment variable feature to hatch ? |
4d96764
to
49c0a3d
Compare
No I actually mean to copy the concept over here. Someone opened up a PR implementing this same solution (it's very logical) but then I thought about how it would play out in practice so I came up with that way pypa/hatch#1455 (comment) |
Oh, okay I see! Thanks for the suggestion! |
49c0a3d
to
9668d79
Compare
Hi @ofek, I upaded the PR. Is this what you had in mind? 🙂 I have to update the documentation |
9668d79
to
2809c6f
Compare
Updated the variable name to Have stuff to do, I will update very soon the doc for the UV customization. |
2809c6f
to
8126616
Compare
At a glance this is exactly what I had in mind, thanks! I will take a look this weekend. |
This enables enterprise use-cases.
8126616
to
0e1c5e1
Compare
It looks like your branch doesn't allow pushes, I'm going to create a new branch with your commits #152 |
Superseded by #153 |
Fixes #146
I added also some tests ensuring the non-regression 🙂