-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: add exports #1044
feat: add exports #1044
Conversation
@@ -1,5 +1,5 @@ | |||
ARG NODE_VERSION=18 | |||
FROM node:${NODE_VERSION}-alpine3.18 |
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.
The docker image were using is deprecated now. I think this is the cause of the font changes in all the snapshots?
ifeq ($(USE_LEGACY), true) | ||
dockerfile = -f Dockerfile.legacy | ||
endif | ||
|
||
ifeq ($(USE_LEGACY), true) | ||
dockerfile = -f Dockerfile.legacy | ||
endif | ||
|
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.
The 'legacy' Dockerfile was for supporting node v18. node 18 is very close to end of life and we weren't even running tests against it anymore.
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.
🎉
This PR was released!🚀 Changes included in v7.12.0 |
🧰 Changes
Adds support for multiple exports.
Prior to this, we were simply ignoring any non-default exports. This pulls the exports from the passed in components, and adds them to the hook that provides component lookup.
🧬 QA & Testing