-
-
Notifications
You must be signed in to change notification settings - Fork 629
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
The sphinx inventory files in readthedocs is missing top-level symbols #2739
Comments
This is the appropriate place. Thanks for investigating and reporting. |
This is required since the latest marshmallow release, until a fix is available for: marshmallow-code/marshmallow#2739 Signed-off-by: Sahas Subramanian <[email protected]>
@lafrech thanks for the quick reply. Do you think this could take long to fix? We need to decide if we go for a workaround or if we wait for the fix. Thanks! |
i'll take a look at this now. looks like this happened in #2713 . should be a quick fix |
This is because of the latest marshmallow release missing symbols causing mkdoc builds to fail. This needs to be reverted once this is resolved: marshmallow-code/marshmallow#2739. Signed-off-by: Sahas Subramanian <[email protected]>
fixed in 3.x-line and latest in the docs. |
@llucax can you verify your docs are working now? |
Hi @sloria, sorry for the delay, I can confirm it works again with |
great! it's also fixed for the |
We are using the sphinx inventory files hosted in readthedocs for cross referencing of our docs.
Today we noticed the build is failing because the inventory files are missing top-level symbols (i.e. symbols re-exported by
marshmallow.__init__
). For example:This used to work, and I noticed there was a new release yesterday so I was wondering if anything changed in terms of how these files are generated.
I checked the inventory file and they are really not there:
I'm sorry if this is not the appropriate place to report this and please let me know where I should do it if it is not here.
The text was updated successfully, but these errors were encountered: