-
Notifications
You must be signed in to change notification settings - Fork 620
chore: Delete the toggleStyledComponent utility we used for CSS modules feature flags #6308
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
Conversation
|
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.
Pull Request Overview
This PR removes the toggleStyledComponent
utility function and its associated tests, which were previously used for CSS modules feature flag toggles but are no longer needed.
- Deletes the
toggleStyledComponent
utility function that was used to conditionally render styled components based on feature flags - Removes the corresponding test file that validated the utility's functionality
- Cleans up legacy code from the CSS modules feature flag implementation
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
File | Description |
---|---|
packages/react/src/internal/utils/toggleStyledComponent.tsx |
Complete removal of the utility function that handled conditional rendering of styled vs plain components |
packages/react/src/internal/utils/__tests__/toggleStyledComponent.test.tsx |
Deletion of comprehensive test suite covering all functionality of the removed utility |
👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks! |
size-limit report 📦
|
This deletes the
toggleStyledComponent
utility function we used to toggle when we had the CSS modules feature flags. This is no longer in use anywhereMerge checklist