Skip to content

Commit 39ab87a

Browse files
authored
Enable structured logger ESLint rule for src/archives (#60857)
1 parent 8f20272 commit 39ab87a

1 file changed

Lines changed: 32 additions & 2 deletions

File tree

eslint.config.ts

Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,8 @@ export default [
100100
// Disabled rules to review
101101
'no-console': 'off', // 800+
102102

103-
// Custom rules (disabled by default for now)
104-
'custom-rules/use-custom-logger': 'off',
103+
// Custom rules
104+
'custom-rules/use-custom-logger': 'error',
105105

106106
// Prevent direct res.redirect() usage — use res.safeRedirect() instead
107107
// to avoid open redirect vulnerabilities via protocol-relative URLs.
@@ -169,6 +169,36 @@ export default [
169169
},
170170
},
171171

172+
// Directories not yet migrated to structured logger (see github/docs-engineering#5639)
173+
// Remove directories from this list as they are migrated
174+
{
175+
files: [
176+
'src/ai-tools/**/*.{ts,js}',
177+
'src/article-api/**/*.{ts,js}',
178+
'src/audit-logs/**/*.{ts,js}',
179+
'src/changelogs/**/*.{ts,js}',
180+
'src/color-schemes/**/*.{ts,js}',
181+
'src/content-render/**/*.{ts,js}',
182+
'src/data-directory/**/*.{ts,js}',
183+
'src/dev-toc/**/*.{ts,js}',
184+
'src/events/**/*.{ts,js}',
185+
'src/fixtures/**/*.{ts,js}',
186+
'src/frame/**/*.{ts,js}',
187+
'src/github-apps/**/*.{ts,js}',
188+
'src/journeys/**/*.{ts,js}',
189+
'src/languages/**/*.{ts,js}',
190+
'src/links/**/*.{ts,js}',
191+
'src/metrics/**/*.{ts,js}',
192+
'src/observability/**/*.{ts,js}',
193+
'src/rest/**/*.{ts,js}',
194+
'src/search/**/*.{ts,js}',
195+
'src/shielding/**/*.{ts,js}',
196+
],
197+
rules: {
198+
'custom-rules/use-custom-logger': 'off',
199+
},
200+
},
201+
172202
// Override for scripts, tests, workflows, content-linter, and React files (disable custom logger rule)
173203
{
174204
files: [

0 commit comments

Comments
 (0)