Repository navigation
README: fixes and FAQ answers from an outside read-through - #14
Merged
Merged
Conversation
- Quick start: say it's for a saved script ($PSScriptRoot is empty at the prompt), and skip aborted commands instead of reporting them as failures. - Abort: the numbered steps are for Ctrl+C; a window close skips the wait. New note that the script carries on after an abort, with the `exit` line to stop it. - State the defaults for the batch and runspace-job time limits and concurrency. - Invoke-PkRun: a variable named like one of its own reads its value inside the block unless the block uses .GetNewClosure(). - FAQ: pipeline input, which functions are public, and whether child programs get Ctrl+C. - A "Bugs and questions" section pointing to GitHub issues. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
On a window close the drain is skipped (Windows ends the process within seconds), but the announcement still said 'finishing current jobs, please wait'. It now says jobs are being stopped, matching the README. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Docs only. Fixes from reading the README as a new user would.
What was wrong, and what changed:
$PSScriptRootis empty outside a saved script. A comment now says to use. .\ParallelKit.ps1at the prompt.if ($batch.Aborted) { exit 1 }to stop it.-TimeoutSeconds) weren't stated where the options are introduced.Invoke-PkRun. A variable named like one ofInvoke-PkRun's own ($KeepAwake, say) silently reads the kit's value inside the block. The README now says so, with the.GetNewClosure()fix. Checked by running it.Testing. Docs only, so no harness change. It merges cleanly with #11, #12 and #13.
🤖 Generated with Claude Code