-
Notifications
You must be signed in to change notification settings - Fork 276
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
jupyterlite build fails if --contents dir has any dir with a .
in name
#624
Comments
nice: all's fair in love and reuse when pretending to be a server, i guess. we have a bunch of preset ignored paths, and could add {
"ContentsManager": {
"allow_hidden": true
}
} of note though, some static hosts, no doubt much like us, ignore |
Just getting back to this, @bollwyvl do you think |
To clarify, what is is your use case? Do you want these files, or not want these files indexed (which is what's failing) and just don't want the build to fail.
Not sure, as there are more things at play, and the actual jupyter_server traitlets aren't in play... the above was just an example, and we have to fool the equivlant of
|
@bollwyvl currently the build process just completely crashes if the content directory has any hidden files. I mostly just want it to not crash :) |
@bollwyvl I can't seem to implement your suggestion in #624 (comment) - how would I specify the 'equivalent' of that in config? In |
It probably needs python changes to not break, and schema and potentially
ts changes to actually work... Will have a look.
|
Haven't tried it out with all the overrides, etc. but at least it doesn't asplode. |
@bollwyvl yay! I tried to set that PR up locally to test, but got thwarted by not really being able to understand what is going on and why Thank you very much for working on it, and I'll just test it out when it's out! \o/ |
Welp, when
Such as?
Yerp, at present, there's like 10min of nested python and node stuff happening to get to a working build due to all the layers (e.g. |
@bollwyvl nvm, I got it to work! |
Loosen `.` file detection, add `--extra-ignore-contents`
Description
Running
jupyter lite build --contents contents
fails with the following error if there is any directory whose name starts with a.
.Reproduce
Create a directory named
.binder
, and just put any random file inside it.Expected behavior
Context
Output from
pip list
:The text was updated successfully, but these errors were encountered: