rusackas opened a new pull request, #36431:
URL: https://github.com/apache/superset/pull/36431

   ## Summary
   This PR removes `polished` from direct dependencies since it's never 
imported directly and is already available as a transitive dependency through 
Storybook packages.
   
   ## Analysis
   Dependency analysis reveals:
   - **Zero direct imports**: No imports of `polished` found in src/, 
packages/, or plugins/
   - **Already available transitively**: Included through 
@storybook/addon-actions and @storybook/blocks
   - **No build/config usage**: Not referenced in webpack, babel, or other 
configuration files
   - **Redundant listing**: Having it as both direct and transitive dependency 
is unnecessary
   
   ## Verification
   ```bash
   # No direct imports found
   $ rg "from ['\"]polished|import.*polished|require.*polished" src/ packages/ 
plugins/
   # (no results)
   
   # Still available through Storybook
   $ npm ls polished
   ├─┬ @storybook/[email protected]
   │ └── [email protected]
   ├─┬ @storybook/[email protected]
   │ └─┬ @storybook/[email protected]
   │   └── [email protected] deduped
   ```
   
   ## Changes
   - Removed `polished` (^4.3.1) from package.json direct dependencies
   - Updated package-lock.json accordingly
   - Package remains available through transitive dependencies
   
   ## Impact
   - **Cleaner dependency tree**: Reduces redundant dependencies
   - **No functionality changes**: Library still accessible where needed
   - **Reduced maintenance**: One less direct dependency to track and update
   
   ## Testing
   - Frontend builds successfully
   - Storybook still functions correctly
   - No TypeScript or linting errors
   
   🤖 Generated with [Claude Code](https://claude.ai/code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to