-
Notifications
You must be signed in to change notification settings - Fork 137
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
Enable --only-deps with FLIT_ALLOW_INVALID #631
base: main
Are you sure you want to change the base?
Conversation
@takluyver , have made some updates so that I can use |
Hi @dciborow, sorry it's taken me a while to look at this. I see what you're after, but this feels like the wrong approach to me. Flit is meant to work with a package, and crucial files being missing is not the same thing as metadata that Flit thinks is invalid. The FLIT_ALLOW_INVALID environment variable is basically meant as an 'escape hatch' for bugs in Flit's validation, which is why the docs say:
Maybe @pradyunsg do you have any inspiration here? 🙂 |
There's plans to add |
Currently, trying to run "flit install --only-deps" includes checks that require both the README and the module folder be present, even though they are not actually needed. When trying to use --only-deps in my docker file, I have to also copy the README and have to create a fake directory.
flit already includes a flag to try and install if there are invalid settings,
FLIT_ALLOW_INVALID
. I have extended the use of that flag to enable running --only-deps without the checks. The updated Dockerfile becomes.