haskell / haddock

Haskell Documentation Tool
www.haskell.org/haddock/
BSD 2-Clause "Simplified" License
362 stars 242 forks source link

Use visibility to decide which interfaces are included in quickjump #1490

Closed coot closed 2 years ago

coot commented 2 years ago

This is also consistent with how html index is build. See haskell/cabal#7669 for rationale behind this decision.

Kleidukos commented 2 years ago

:+1:

coot commented 2 years ago

@Kleidukos thanks for merging my small PRs so quickly, I appreciate that - thanks a lot.

btw, I fixed --with-haddock for cabal haddock[s] so I can go around using version of haddock which I want with cabal. That's part of haskell/cabal#8162 right now, although I might re-submit it on its own.

Kleidukos commented 2 years ago

@coot I'll review your PR on cabal when it's ready. :)

coot commented 2 years ago

Thanks @Kleidukos