Enable `no-console` ESLint rule
Created by: valerybugakov
Problem statement
Enable no-console
ESLint rule
Success criteria
-
no-console
is set toerror
in the monorepoeslintrc
. -
no-console
is disabled for server-side code (e.g.,client/web/dev
orgulpfile.js
) and Storybook stories viaeslintrc
overrides
field. -
no-console
is disabled forbrowser
andVSCode
packages in theireslintrc
configs. - the new
logError
utility function is added to the@sourcegraph/common
package.export function logError(...args: unknown[]): void { // eslint-disable-next-line no-console console.error(...args) }
-
console.error()
calls are replaced with the newlogError
utility function. -
console.log()
calls are removed or replaced with the newlogError
utility function.
Time estimate
- This task should take 3 hours at maximum. Ping the reviewer in the spec pull request if time-consuming changes are required.