Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc node:process fixes #14621

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

misc node:process fixes #14621

wants to merge 26 commits into from

Conversation

nektro
Copy link
Contributor

@nektro nektro commented Oct 16, 2024

  • fixes a bug where padding multiple "ipc" options to node:child_process.spawn stdio option did not throw an error
  • add definition and implementation for process.availableMemory()
  • fixes a bug where passing process.chdir a non-string non-undefined argument would not throw an error
  • add definition and implementation for process.constrainedMemory()
  • fixes error handling of process.cpuUsage()
  • fixes error handling of process.hrtime()
  • fixes a bug where process.exit() was not calling process.reallyExit()
  • fixes fields of process.release; removes .lts, fills out proper content of .headersUrl, removes bad field .libUrl
  • add definition and implementation for process.setegid(), .seteuid(), .setgid(), .setuid(), .setgroups()

@nektro nektro marked this pull request as ready for review October 17, 2024 23:12
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