Fix use of `@internal` breaking exported typing

graphile / crystal

🔮 Graphile's Crystal Monorepo; home to Grafast, PostGraphile, pg-introspection, pg-sql2 and much more!

Other

12.49k stars 568 forks source link

Fix use of @internal causing invalid d.ts files (fixes https://github.com/graphile/crystal/issues/2093)

add types-test using tsafe
ensure types required in exported interfaces are not @internal

Description

Performance impact

None

Security impact

None

Checklist

[x] My code matches the project's code style and yarn lint:fix passes.
[x] I've added tests for the new feature, and yarn test passes.
[x] I have detailed the new feature in the relevant documentation.
[ ] I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
[ ] If this is a breaking change I've explained why.

🦋 Changeset detected

Latest commit: 7bb1573ba45a4d8b7fa9ad53cdd79686d2641383

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 15 packages

| Name | Type | | -------------------------- | ----- | | grafast | Patch | | @localrepo/grafast-bench | Patch | | @dataplan/json | Patch | | @dataplan/pg | Patch | | @grafserv/persisted | Patch | | grafserv | Patch | | ruru-components | Patch | | @localrepo/grafast-website | Patch | | graphile-build-pg | Patch | | graphile-build | Patch | | graphile-utils | Patch | | pgl | Patch | | postgraphile | Patch | | graphile-export | Patch | | graphile | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

graphile / crystal