What would you like to improve?
The frontend currently enforces zero lint warnings/errors, but build warnings were still allowed. We should align all thresholds to zero and remove the remaining known build warnings so CI reliably blocks regressions
What's the current friction?
Current state:
LINT_WARNING_THRESHOLD=0
LINT_ERROR_THRESHOLD=0
BUILD_WARNING_THRESHOLD=24
Known build warnings:
ExternalDocLinkComponent is imported but unused in GlobalPreferencesComponent.
- CommonJS warning for
@stomp/stompjs used by @stomp/rx-stomp.
What would make this better?
- Remove unused
ExternalDocLinkComponent import/registration from GlobalPreferencesComponent
- Add
@stomp/stompjs to allowedCommonJsDependencies in Angular build config
- Set
BUILD_WARNING_THRESHOLD=0 in .github/workflows/angular-lint-threshold.yml
Anything else? (Optional)
No response
Before Submitting
What would you like to improve?
The frontend currently enforces zero lint warnings/errors, but build warnings were still allowed. We should align all thresholds to zero and remove the remaining known build warnings so CI reliably blocks regressions
What's the current friction?
Current state:
LINT_WARNING_THRESHOLD=0LINT_ERROR_THRESHOLD=0BUILD_WARNING_THRESHOLD=24Known build warnings:
ExternalDocLinkComponentis imported but unused inGlobalPreferencesComponent.@stomp/stompjsused by@stomp/rx-stomp.What would make this better?
ExternalDocLinkComponentimport/registration fromGlobalPreferencesComponent@stomp/stompjstoallowedCommonJsDependenciesin Angular build configBUILD_WARNING_THRESHOLD=0in.github/workflows/angular-lint-threshold.ymlAnything else? (Optional)
No response
Before Submitting