-
Notifications
You must be signed in to change notification settings - Fork 2k
chore: add wildcard to globals export #20866
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
chore: add wildcard to globals export #20866
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #20866 +/- ##
==========================================
+ Coverage 85.53% 92.39% +6.86%
==========================================
Files 357 511 +154
Lines 14475 37352 +22877
Branches 4818 5741 +923
==========================================
+ Hits 12381 34511 +22130
- Misses 1956 2693 +737
- Partials 138 148 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c99c31c
We are running into issues around pulling in the
icon-loader.jsfrom theglobals/internalfolder and our application complains that it's not able to find the export. Adding the wildcard to thees/globalsexport field in the @carbon/web-components package.json - aligning with thees/componentsexport, fixes this issue for us.Changelog
Changed
PR Checklist
As the author of this PR, before marking ready for review, confirm you:
[ ] Updated documentation and storybook examples[ ] Wrote passing tests that cover this change[ ] Addressed any impact on accessibility (a11y)[ ] Tested for cross-browser consistency[ ] Validated that this code is ready for review and status checks should passMore details can be found in the pull request guide