-
Notifications
You must be signed in to change notification settings - Fork 511
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
Support non-library projects #1119
Comments
Closing this out as I just saw while I was working on this, you fixed the issue in the |
I'm reopening it because it's better to fix the lib for other non-library projects. The issues I found were:
|
I just ran into this same issue while making a gradual onboarding guide for my users. So I would love this feature. |
@dleen I would love to take a look at the onboarding guide if possible! I'll try and tackle this soon |
cc: @jph00
The fastai/apl-study repo no longer builds the website due do
nbdev_docs
failing. I looked into it, and have am easy workaround. I think it'd be better to fix in the main library, if this is a desired use case for nbdev. I'd love to work on the PR, but wanted to document the issue and get OK on direction first.The Explanation
nbdev_docs
now requires a_modidx.py
file. For anything that isn't a python library that does not exist. Whennbdev_docs
is called, it attempts to create themodidx.py
file vianbdev.doclinks._build_modidx
. This fails because thelib_path
directory defined in thesettings.ini
does not exist because it's not a library so it was never created. If you create that directory as an empty directory withmkdir
the docs build correctly.Question
The first question is: Is this use case is something nbdev should support, or if applications where there is no python library would be better suited for Quarto in the future?
Possible Solutions
I am happy to submit a PR for either of these solutions below, or another approach if you have other ideas.
The fastest workaround would be to have a workaround would be for
_build_modidx
to createlib_path
if it does not exist. I think a better solution would be to modify the process so that a_modidx
is not required if there is no library as part of the nbdev project.Details
Recreate the issue
nbdev_docs
Workaround
mkdir apl-study
nbdev_docs
The text was updated successfully, but these errors were encountered: