-
Notifications
You must be signed in to change notification settings - Fork 109
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
Use named exports across codebase #976
Conversation
704b5b7
to
293a941
Compare
a1286b1
to
8053454
Compare
8053454
to
485a9e1
Compare
export { default as useDeepCompareState } from './useDeepCompareState'; | ||
export { default as useSingleLineTemplateEvaluation } from './useSingleLineTemplateEvaluation'; | ||
export { default as useTemplateEvaluation } from './useTemplateEvaluation'; | ||
export { useCondition } from './useCondition'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI in the future we should reconsider the use of these barrel files. They are very bad for performance and makes tree-shaking harder/impossible
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely worth an investigation at the very least. But it's pretty much the standard across all bpmn.io libs so we might want to discuss it with at least Nico.
Closes #918