Skip to content

Commit

Permalink
fix: Remove extra threshold reporting to clean up default builds. (#219)
Browse files Browse the repository at this point in the history
Since the threshold reporting doesn't really give anything actionable move it behind the normal profiling flag.
  • Loading branch information
Zoramite committed Apr 21, 2023
1 parent 6b1d19c commit 48f90e7
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions packages/amagaki/src/profile.ts
Original file line number Diff line number Diff line change
Expand Up @@ -251,13 +251,12 @@ export class ProfileReport {
}

output(showExpandedReport = false) {
const shownTimerKeys: Set<string> = new Set();
this.reportThreshold(shownTimerKeys);

if (!showExpandedReport) {
return;
}

const shownTimerKeys: Set<string> = new Set();
this.reportThreshold(shownTimerKeys);
this.reportHooks(shownTimerKeys);
this.reportTimers(shownTimerKeys);
this.reportSlowBuilds(shownTimerKeys);
Expand Down

0 comments on commit 48f90e7

Please sign in to comment.