Remove unused tailwindcss devDependency - #516
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the tailwindcss UI devDependency (and its lockfile entry) based on an audit showing Tailwind is not currently loaded/used in the UI build.
Changes:
- Removed
tailwindcssfromawx/ui/package.jsondevDependencies. - Removed
tailwindcssfromawx/ui/package-lock.json(dependency list andnode_modules/tailwindcssentry).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| awx/ui/package.json | Drops the tailwindcss devDependency from the UI toolchain dependencies. |
| awx/ui/package-lock.json | Updates the npm lockfile to remove the resolved Tailwind package and related metadata. |
Files not reviewed (1)
- awx/ui/package-lock.json: Generated file
Comments suppressed due to low confidence (1)
awx/ui/package-lock.json:109
- The UI still uses an
npmlockfile (package-lock.json). Project guidance preferspnpmfor Node tooling; if that’s the intended direction, consider migrating this workspace topnpm(addpnpm-lock.yaml, update install/build docs and CI, and removepackage-lock.json) rather than continuing to update the npm lockfile.
Comment on lines
101
to
106
| "resolve-url-loader": "^5.0.0", | ||
| "sass-loader": "^16.0.8", | ||
| "semver": "^7.8.0", | ||
| "style-loader": "^4.0.0", | ||
| "styled-components": "^6.4.2", | ||
| "tailwindcss": "^4.3.0", | ||
| "terser-webpack-plugin": "^5.6.0", |
Contributor
There was a problem hiding this comment.
Yes, lets strip the relevant portions out of the webpack config also, just to finish cleaning up.
tailwindcss is referenced in webpack.config.js behind a useTailwind guard that checks for tailwind.config.js, which does not exist. No component uses tailwind classes, @apply, or any tailwind directive. The package is installed but never loaded.
Strip the useTailwind guard and Tailwind-specific PostCSS plugin branch from the webpack config, completing the tailwindcss removal.
blaipr
force-pushed
the
remove/tailwindcss
branch
from
June 25, 2026 21:10
0b5e9ed to
a1c52de
Compare
Contributor
Author
|
Fixed. Rebased on main and stripped the Tailwind branch from the webpack config. |
cigamit
approved these changes
Jun 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
SUMMARY
Remove tailwindcss, which is installed but never loaded.
useTailwindguardthat checks for
tailwind.config.js, which does not existISSUE TYPE
COMPONENT NAME
ASCENDER VERSION
ADDITIONAL INFORMATION
Identified by auditing npm devDependencies for unused packages.