build(biome): enable noConsoleLog rule

[setchy]

Avoid future console.logs accidentally creeping in 🙈

[afonsojramos]

The fact that you refactored a console log into its own function to not put the ignore on multiple places is a bit hilarious 😆

[setchy]

The fact that you refactored a console log into its own function to not put the ignore on multiple places is a bit hilarious 😆

just making our friend @bmulholland happy 😄

#1268 (comment)

[bmulholland]

Well, to be fair, I was thinking of going one step further:
logger.ts:

function debugLog(msg) {
  // biome-ignore lint/suspicious/noConsoleLog: debug logging
  console.log(msg);
}

The point isn't to centralize the biome-ignore, but more to make the intent clear. Specifically: if I'm reading the code, and I see console.log, I think "this is a mistake, shouldn't be here." But if I'm reading code and see debugLog, I don't question it, and even consider it thoughtful and helpful.

Also, centralizing it means that we could in future e.g. change the debug log to a log file, which could be submitted with bug reporst.

My mental reference here is Rails' log levels: https://guides.rubyonrails.org/debugging_rails_applications.html#sending-messages