Skip to content

chore: suppress PostCSS warning and enhance logger #36

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.

Sign up for GitHub

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

Merged
bniladridas merged 6 commits into from
Nov 22, 2025
Merged

chore: suppress PostCSS warning and enhance logger #36

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
e92645b
chore: improve logger and clean warnings
bniladridas Nov 22, 2025
d461957
chore: add postcss config to resolve warnings
bniladridas Nov 22, 2025
317b6e8
refactor: add type check to postcss warning suppression
bniladridas Nov 22, 2025
8ec01a5
refactor(types): replace `any` error with `httperror` interface
bniladridas Nov 22, 2025
8ad624d
refactor(logging): extract `getcurrentloglevelvalue` helper to remove…
bniladridas Nov 22, 2025
d213405
refactor(logging): remove duplicate log level logic
bniladridas Nov 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions apps/server/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,14 +47,13 @@ app.use((req, res, next) => {
const configuredApp = registerRoutes(app);

configuredApp.use(
/* eslint-disable @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any */
// eslint-disable-next-line @typescript-eslint/no-explicit-any
(err: any, _req: Request, res: Response, _next: NextFunction) => {
const status = err.status || err.statusCode || 500;
const message = err.message || 'Internal Server Error';

res.status(status).json({ message });
}
/* eslint-enable @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any */
);

// importantly only setup vite in development and after
Expand Down
15 changes: 12 additions & 3 deletions eslint.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,10 @@ export default [
},
rules: {
...tseslint.configs.recommended.rules,
'@typescript-eslint/no-unused-vars': 'error',
'@typescript-eslint/no-unused-vars': [
'error',
{ argsIgnorePattern: '^_' },
],
'@typescript-eslint/no-explicit-any': 'warn',
},
},
Expand All @@ -47,7 +50,10 @@ export default [
},
rules: {
...tseslint.configs.recommended.rules,
'@typescript-eslint/no-unused-vars': 'error',
'@typescript-eslint/no-unused-vars': [
'error',
{ argsIgnorePattern: '^_' },
],
'@typescript-eslint/no-explicit-any': 'warn',
},
},
Expand All @@ -69,7 +75,10 @@ export default [
},
rules: {
...tseslint.configs.recommended.rules,
'@typescript-eslint/no-unused-vars': 'error',
'@typescript-eslint/no-unused-vars': [
'error',
{ argsIgnorePattern: '^_' },
],
'@typescript-eslint/no-explicit-any': 'warn',
},
},
Expand Down
Loading