Skip to content

Commit

Permalink
chore: remove debugging console output (#74)
Browse files Browse the repository at this point in the history
This also adds a rule to *warn* about doing so in the distro lib files.
I don't want to error on these because it I think it would get in the
way of PR dev. If a console.log slips in, then that's not so bad.
  • Loading branch information
trentm authored Feb 13, 2024
1 parent adb10c1 commit 45d0cb5
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
5 changes: 5 additions & 0 deletions packages/opentelemetry-node/lib/.eslintrc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"rules": {
"no-console": "warn"
}
}
3 changes: 0 additions & 3 deletions packages/opentelemetry-node/lib/metrics/host.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,9 +172,6 @@ class SystemCpuUtilizationAggregator {
accumulationByAttributes,
endTime
) {
console.log('toMetricdata');
console.dir(accumulationByAttributes, {depth: 5});

// We cannot sum up the utilization of all the states since `os.cpus()` is
// not returning all of the possible states but limited to: user, nice, sys, idle, irq
// https://nodejs.org/api/all.html#all_os_oscpus
Expand Down

0 comments on commit 45d0cb5

Please sign in to comment.