Skip to content
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

Fix spurious "Package.Package missing" warnings #1009

Merged
merged 2 commits into from
May 8, 2019

Conversation

mortenpi
Copy link
Member

@mortenpi mortenpi commented May 6, 2019

It seems that going from Julia 0.6 to 0.7 the way module bindings are stored in the docs metadata changed from Binding(Mod, :SubMod) to Binding(Mod.SubMod, :SubMod). Documenter, however, still constructs the former Binding objects for modules that are listed just as SubMod in an at-docs block.

This fixes the spurious errors that causes when checking for missing docs by explicitly normalizing the module bindings that Documenter creates before it crosschecks them against the bindings in docs metadata.

It seems that going from Julia 0.6 to 0.7 the way module bindings are
stored in the docs metadata changed from Binding(Mod, :SubMod) to
Binding(Mod.SubMod, :SubMod). Documenter, however, still constructs the
former Binding objects for modules that are listed just as SubMod in an
at-docs block.

This fixes the spurious errors that causes when checking for missing
docs by explicitly normalizing the module bindings that Documenter
creates before it crosschecks them against the bindings in docs
metadata.
@mortenpi mortenpi added this to the 0.22.4 milestone May 7, 2019
@mortenpi mortenpi merged commit 004463c into master May 8, 2019
@mortenpi mortenpi deleted the mp/fix-missing-modules branch May 8, 2019 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant