Skip to content

Feat/depcheck#496

Open
cedric07 wants to merge 6 commits intomasterfrom
feat/depcheck
Open

Feat/depcheck#496
cedric07 wants to merge 6 commits intomasterfrom
feat/depcheck

Conversation

@cedric07
Copy link
Copy Markdown
Contributor

@cedric07 cedric07 commented Apr 28, 2026

Voir ici : https://www.notion.so/beapi/Test-Depcheck-C-dric-Avril-2026-3500865393b9809da840dcdb6fe429eb


Note

Medium Risk
Touches the build toolchain (Webpack sass-loader resolution, SCSS imports, and devDependencies), which could break styling/builds if resolution differs between environments. No runtime/application logic changes beyond asset pipeline behavior.

Overview
Adds depcheck tooling via a new .depcheckrc ignore list and a yarn depcheck script to better validate dependency usage in this Webpack/Stylelint-based setup.

Updates the SCSS pipeline to support package-style Sass imports by adding sass-loader loadPaths and switching Fontsource @use statements to module paths; also removes style-loader from devDependencies and simplifies webpack-theme-json-plugin logging by dropping chalk.

Reviewed by Cursor Bugbot for commit f90784f. Bugbot is set up for automated code reviews on this repo. Configure here.

@cedric07 cedric07 requested a review from francoistibo April 28, 2026 13:33
@cedric07 cedric07 changed the base branch from master to fix/node-modules-import April 28, 2026 13:36
@cedric07 cedric07 changed the base branch from fix/node-modules-import to master April 28, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant