Merge cardano-api:internal with cardano-api main library under Cardano.Api.Internal.* and export required symbols

[palas]

Changelog

- description: |
    Merged `cardano-api:internal` with `cardano-api` main library under `Cardano.Api.Internal.*` and expose requried symbols
  type:
  - refactoring    # QoL changes

Context

Links between different components of a project in Haddock do not seem to work. Documentation in #698 link documentation from modules in cardano-api with modules in cardano-api:internal. For that reason, this PR combines the internal component with the main library component.

How to trust this PR

Check no functionality was modified. Ensure the organisation of modules and re-exports makes sense.

I suggest considering the result of not applying the last commit (keeping the modules separated inside the Cardano.Api.Internal. Which may be clearer.

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• New tests are added if needed and existing tests are updated. See Running tests for more details
• Self-reviewed the diff

[smelc]

I'm not the one who should approve, but I think this is a great change 👍

[carbolymer]

Besides my other remarks, nice change!

[carbolymer]

Links between different components of a project in Haddock do not seem to work. Documentation in #698 link documentation from modules in cardano-api with modules in cardano-api:internal

What do you mean here? I'm able to jump between cardano-api modules just fine here: https://cardano-api.cardano.intersectmbo.org/ Can you provide examples? If linking is still broken, can you add the examples to #601 to not lose track of them

[Jimbo4350]

LGTM! I agree with @carbolymer's comments

[palas]

What do you mean here? I'm able to jump between cardano-api modules just fine here: https://cardano-api.cardano.intersectmbo.org/ Can you provide examples? If linking is still broken, can you add the examples to #601 to not lose track of them

It is when you write a specific link to something in a comment, by using ", not when it is implied by the actual code. You can see the example here: https://github.com/IntersectMBO/cardano-api/pull/698/files#diff-a0184298d1665714978e4d3d3400bea78e52eb231211cefea8da26a0ed3cd4deR9