chore(eslint): no-console error rule

[skjindal93]

Which problem is this PR solving?

Adds ESLint rule for avoiding any console log statements in the codebase
Fixes #2007 and #1861

[codecov]

Codecov Report

Merging #2018 (03a96af) into main (6025749) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 03a96af differs from pull request most recent head 36eff7b. Consider uploading reports for the commit 36eff7b to get more accurate results

@@           Coverage Diff           @@
##             main    #2018   +/-   ##
=======================================
  Coverage   92.97%   92.98%           
=======================================
  Files         152      152           
  Lines        5925     5926    +1     
  Branches     1245     1245           
=======================================
+ Hits         5509     5510    +1     
  Misses        416      416           

Impacted Files	Coverage Δ
...emetry-metrics/src/export/ConsoleMetricExporter.ts	62.50% <ø> (ø)
...elemetry-tracing/src/export/ConsoleSpanExporter.ts	78.57% <ø> (ø)
...ackages/opentelemetry-web/src/WebTracerProvider.ts	100.00% <100.00%> (ø)

[Flarna]

I think the console exporters should actually export to console.

[skjindal93]

I think the console exporters should actually export to console.

I had the same dilemma. How about we add the following in the constructor of console exporters?

diag.setLogger(new DiagConsoleLogger(), DiagLogLevel.INFO);

[Flarna]

This will redirect all logs not just the exports.

I would disable the rule for the few places in the exporter where console is used. But maybe wait a little before you adapt the PR as there could be other options :o).

[skjindal93]

@vmarchaud @Flarna @legendecas Can you please merge the PR, since I don't have the write access to the repository?

[Flarna]

@skjindal93 Two maintainer approvals are required in this repo but there is only one till now.

[obecny]

@skjindal93 did you sign the CLA ?

[dyladan]

Close and reopen to retrigger CLA