-
Notifications
You must be signed in to change notification settings - Fork 42
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
Reduce bundle size (7.2MB -> 5.4MB) with consistent grommet imports #1038
Conversation
MegaLinter status: ✅ SUCCESS
See errors details in artifact MegaLinter reports on CI Job page |
Considering the current build system, does using "import type" (instead of simple import) make any difference in the compiled product? I see a mixed practice in the current codebase, including this PR. |
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.
I see no issues, besides the small thing of inconsistent usage of import type
.
Ah yeah, I'll remove that import type (has nothing to do with reducing size) |
b4a73e2
to
70b4ad0
Compare
internals/jest/jest.base-config.js
Outdated
@@ -18,7 +18,7 @@ const config = { | |||
transform: { | |||
'^.+\\.(js|jsx|mjs|cjs|ts|tsx)$': '<rootDir>/internals/jest/babelTransform.js', | |||
}, | |||
transformIgnorePatterns: ['/node_modules/(?!(cborg)/)'], | |||
transformIgnorePatterns: ['/node_modules/(?!(cborg)|(grommet/es6)|(grommet-icons/es6)/)'], |
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.
I would go with one capturing group
transformIgnorePatterns: ['/node_modules/(?!(cborg|grommet/es6|grommet-icons/es6)/)'],
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.
yep yep
70b4ad0
to
b219841
Compare
b219841
to
5f9fae6
Compare
No description provided.