-
Notifications
You must be signed in to change notification settings - Fork 944
Description
Originally posted by @LisaFC in #535 (reply in thread):
Correct - feature is in but disabled by default, docs have not been updated yet. Anyone who wants to try it can use the preview for the docs PR for more info. The docs update is sizeable and I want to take some time to do some organization work on it.
Based on the above comment, I believe that it wasn't @LisaFC's intention to merge #801 and introduce such a serious breaking change: #801 broke the builds of most users.
I believe that our first order of business is to (at least partially) revert #801 so that all users can continue to work from HEAD, as we continue to work through the details of how to introduce modules as an optional feature (cf. #848).
We should also restore the build of the user-guide to be submodule based, since that reflects how most users are including docsy into their projects, and as such, it acts as a test that docsy@HEAD hasn't broken anything for submodule users.