Skip to content

Conversation

JakeChampion
Copy link
Contributor

@JakeChampion JakeChampion commented Mar 19, 2025

The process.cpuUsage method when called with a parameter, is meant to compute the diff between the current cpu-usage and the supplied cpu-usage values

This should make /tests/node_compat/test/pummel/test-process-cpuUsage.js begin to pass

@JakeChampion JakeChampion marked this pull request as ready for review March 19, 2025 14:23
@JakeChampion JakeChampion force-pushed the patch-2 branch 13 times, most recently from adfbb55 to 1c454ac Compare March 25, 2025 10:43
@JakeChampion JakeChampion force-pushed the patch-2 branch 2 times, most recently from ccad420 to 17877eb Compare March 25, 2025 14:52
@ry ry requested a review from kt3k March 31, 2025 12:45
@JakeChampion JakeChampion force-pushed the patch-2 branch 2 times, most recently from 37178c4 to 1bc62ff Compare April 1, 2025 08:18
@kt3k kt3k changed the title fix: support the optional previousValue parameter for process.cpuUsage() fix(ext/node): support the optional previousValue parameter for process.cpuUsage() Apr 1, 2025
Copy link
Member

@kt3k kt3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice! Thanks

@kt3k kt3k merged commit 0f40ee7 into denoland:main Apr 1, 2025
18 checks passed
@JakeChampion JakeChampion deleted the patch-2 branch April 1, 2025 14:34
littledivy pushed a commit that referenced this pull request Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants