docs: add utility process info to tutorial docs by deepak1556 · Pull Request #36074 · electron/electron · GitHub
Skip to content

docs: add utility process info to tutorial docs#36074

Merged
zcbenz merged 1 commit into
mainfrom
robo/update_tutorial_for_utility_process
Oct 27, 2022
Merged

docs: add utility process info to tutorial docs#36074
zcbenz merged 1 commit into
mainfrom
robo/update_tutorial_for_utility_process

Conversation

@deepak1556

Copy link
Copy Markdown
Member

Description of Change

Follow-up to #34980 (comment)

Release Notes

Notes: no-notes

@deepak1556 deepak1556 requested a review from erickzhao October 19, 2022 14:22
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Oct 19, 2022
@patchup patchup Bot requested a review from a team as a code owner October 19, 2022 14:36
Comment thread docs/tutorial/process-model.md Outdated
Base automatically changed from robo/feat_enable_utility_process_api to main October 20, 2022 05:49
@electron-cation electron-cation Bot added documentation 📓 semver/patch backwards-compatible bug fixes labels Oct 20, 2022
@deepak1556 deepak1556 force-pushed the robo/update_tutorial_for_utility_process branch from 3d25f5a to 4add784 Compare October 20, 2022 07:57
@deepak1556 deepak1556 requested review from erickzhao and removed request for a team October 20, 2022 07:57
@deepak1556 deepak1556 removed the semver/patch backwards-compatible bug fixes label Oct 20, 2022
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Oct 20, 2022
@zcbenz zcbenz merged commit 260678b into main Oct 27, 2022
@zcbenz zcbenz deleted the robo/update_tutorial_for_utility_process branch October 27, 2022 11:55
@release-clerk

release-clerk Bot commented Oct 27, 2022

Copy link
Copy Markdown

deepak1556 added a commit that referenced this pull request Nov 17, 2022
* feat: UtilityProcess API (#34980)

* chore: initial scaffolding

* chore: implement interface and docs

* chore: address code style review

* fix: cleanup of utility process on shutdown

* chore: simplify NodeBindings::CreateEnvironment

* chore: rename disableLibraryValidation => allowLoadingUnsignedLibraries

* chore: implement process.parentPort

* chore(posix): implement stdio pipe interface

* chore(win): implement stdio interface

* chore: reenable SetNodeOptions for utility process

* chore: add specs

* chore: fix lint

* fix: update kill API

* fix: update process.parentPort API

* fix: exit event

* docs: update exit event

* fix: tests on linux

* chore: expand on some comments

* fix: shutdown of pipe reader

Avoid logging since it is always the case that reader end of
pipe will terminate after the child process.

* fix: remove exit code check for crash spec

* fix: rm PR_SET_NO_NEW_PRIVS for unsandbox utility process

* chore: fix incorrect rebase

* fix: address review feedback

* chore: rename utility_process -> utility

* chore: update docs

* chore: cleanup c++ implemantation

* fix: leak in NodeServiceHost impl

* chore: minor cleanup

* chore: cleanup JS implementation

* chore: flip default stdio to inherit

* fix: some api improvements

* Support cwd option
* Remove path restriction for modulePath
* Rewire impl for env support

* fix: add tests for cwd and env option

* chore: alt impl for reading stdio handles

* chore: support message queuing

* chore: fix lint

* chore: new UtilityProcess => utilityProcess.fork

* fix: support for uncaught exception exits

* chore: remove process.execArgv as default

* fix: windows build

* fix: style changes

* fix: docs and style changes

* chore: update patches

* spec: disable flaky test on win32 arm CI

Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>

Co-authored-by: Robo <hop2deep@gmail.com>

* chore: update patches

* docs: add utility process info to tutorial docs (#36074)

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Robo <hop2deep@gmail.com>
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants