Skip to content

Commit 28dd5ce

Browse files
small quality of live improvements (#1489)
- ignore next-env.d.ts from eslint - it's throwing errors but shouldn't be modified by the user - suppress next upgrade logging changes, currently it logs all api calls but this isn't necessary because pino logs on the backend
1 parent 55e1754 commit 28dd5ce

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

packages/app/eslint.config.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ export default [
1515
prettierPlugin,
1616
{
1717
ignores: [
18+
'next-env.d.ts',
1819
'playwright-report/**',
1920
'.next/**',
2021
'node_modules/**',

packages/app/next.config.mjs

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const __dirname = dirname(__filename);
99

1010
// Read version from package.json
1111
const packageJson = JSON.parse(
12-
readFileSync(join(__dirname, 'package.json'), 'utf-8')
12+
readFileSync(join(__dirname, 'package.json'), 'utf-8'),
1313
);
1414
const { version } = packageJson;
1515

@@ -51,7 +51,10 @@ const nextConfig = {
5151
// TODO: Re-evaluate when Turbopack CSS module support improves
5252
// Ignore otel pkgs warnings
5353
// https://github.com/open-telemetry/opentelemetry-js/issues/4173#issuecomment-1822938936
54-
webpack: (config, { buildId, dev, isServer, defaultLoaders, nextRuntime, webpack }) => {
54+
webpack: (
55+
config,
56+
{ buildId, dev, isServer, defaultLoaders, nextRuntime, webpack },
57+
) => {
5558
if (isServer) {
5659
config.ignoreWarnings = [{ module: /opentelemetry/ }];
5760
}
@@ -78,7 +81,12 @@ const nextConfig = {
7881
}
7982
: {}),
8083
}),
84+
logging: {
85+
incomingRequests: {
86+
// We also log this in the API server, so we don't want to log it twice.
87+
ignore: [/\/api\/.*/],
88+
},
89+
},
8190
};
8291

8392
export default nextConfig;
84-

0 commit comments

Comments
 (0)