Skip to content

Config documentation improvements #866

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dobesv
Copy link

@dobesv dobesv commented Mar 17, 2025

There are a lot of features and options that are not described on the website, and can only be found be perusing the source code. This attempts to add some of that to the docs in the reference page.

@@ -79,7 +79,7 @@ export class WorkerRunner implements TargetRunner {
const scriptFile = target.options?.worker ?? target.options?.script;

if (!scriptFile) {
throw new Error('WorkerRunner: "script" configuration is required - e.g. { type: "worker", script: "./worker.js" }');
throw new Error('WorkerRunner: "worker" configuration is required - e.g. { type: "worker", options: { worker: "path/to/worker.js" } }');
Copy link
Author

Choose a reason for hiding this comment

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

Updated this message since it seems like "worker" is the preferred key name now per the examples already provided in the docs.

@dobesv dobesv changed the title Try to improve the docs Config documentation improvements Mar 17, 2025
@dobesv dobesv force-pushed the config-docs-improvements branch 3 times, most recently from d1c3ab6 to 00c560a Compare April 1, 2025 21:24
dobesv added 2 commits April 18, 2025 21:16
There are a lot of features and options that are not described on the website, and can only be found be perusing the source code.  This attempts to add some of that to the docs in the reference page.
@dobesv dobesv force-pushed the config-docs-improvements branch from 00c560a to 020701e Compare April 19, 2025 04:16
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.

1 participant