Skip to content

update exports in main.ts to avoid exporting internal utilities/types#741

Merged
stackoverfloweth merged 1 commit intomainfrom
more-explicit-exports
Mar 19, 2026
Merged

update exports in main.ts to avoid exporting internal utilities/types#741
stackoverfloweth merged 1 commit intomainfrom
more-explicit-exports

Conversation

@stackoverfloweth
Copy link
Copy Markdown
Contributor

update exports to avoid API docs generation for things that aren't intended to be public

this issue was flagged in this auto-generated docs PR

@stackoverfloweth stackoverfloweth self-assigned this Mar 19, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 19, 2026

Deploy Preview for kitbag-router ready!

Name Link
🔨 Latest commit 093c701
🔍 Latest deploy log https://app.netlify.com/projects/kitbag-router/deploys/69bc45fcd659c400083e9310
😎 Deploy Preview https://deploy-preview-741--kitbag-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

// Types
export * from './types/createRouteOptions'
export * from './types/hooks'
export type {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense. But I also wonder if we should just add a linting rule about nothing but types in the types folder. Then export everything. Then we can export specific actual code from other directories. Maybe something for future us.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, I like that for future us

@stackoverfloweth stackoverfloweth merged commit 4c3b2a6 into main Mar 19, 2026
6 checks passed
@stackoverfloweth stackoverfloweth deleted the more-explicit-exports branch March 19, 2026 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants