Skip to content

fix(examples): repair dead links and prose typos - #3452

Merged
magurotuna merged 3 commits into
mainfrom
fix/example/links-prose
Aug 18, 2026
Merged

fix(examples): repair dead links and prose typos#3452
magurotuna merged 3 commits into
mainfrom
fix/example/links-prose

Conversation

@magurotuna

Copy link
Copy Markdown
Member
  • Fix 5 dead resource links: class9c typo, module_metadata, Deno.ServeTlsOptions, fresh.deno.dev, codahale.com
  • Canonicalize stale doc URLs (runtime/manual/..., deploy/kv/manual, /examples/* kebab-case, etc.)
  • Fix prose typos: an HTTP, at any time, allows us, Uploading, use the, YAML output example, em-dash -> colon

Replaced links verified live (HTTP 200); deno fmt and deno check pass.

@magurotuna
magurotuna requested a review from avocet-bot August 17, 2026 10:06
@avocet-bot

avocet-bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Avocet review status

State: Completed
Target: 775935437ddb
Requested by: magurotuna
Model: Claude Opus
Attempt: 1 of 3
Updated: 2026-08-17T10:29:39.963Z
Formal review: #3452 (review)

The formal GitHub review is the authoritative result. This comment only reports operational status.

@avocet-bot avocet-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review model: Claude Opus

Model verdict: APPROVE (informational only)

Formal review policy did not authorize an authoritative event (required_checks_missing).

No blocking findings. This is a documentation-only change to @resource links and JSDoc prose across example scripts. I verified every changed link against the repo (which serves docs.deno.com): all eight external doc paths resolve to existing files/anchors — runtime/getting_started/{installation,setup_your_environment}.md, deploy/reference/cron.md, deploy/kv/index.md (old /deploy/kv/manual is now an oldUrl), deploy/classic/queues.md (fixes the class9cclassic typo), deploy/classic/api/runtime-broadcast-channel.md, runtime/reference/cli/bench.md, and the #use-a-node-built-in-module heading anchor in node.md. Internal /examples/ links were corrected to the underscore convention (writing_files, http_server_routing) and to module_metadata_tutorial (confirmed by the tutorial's frontmatter url: and _data.ts); the removed hyphenated dependency-management link pointed to a nonexistent example. Prose fixes (anytime→any time, a HTTP→an HTTP, duplicated "the", Uploadinf→Uploading, hello: word→world, stray * out) are all correct with no leftovers. No code, tests, error handling, or types are affected.

@magurotuna
magurotuna requested a review from avocet-bot August 18, 2026 03:03
@avocet-bot

avocet-bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Avocet review status

State: Completed
Target: 0582bc1db8aa
Requested by: magurotuna
Model: Claude Opus
Attempt: 1 of 3
Updated: 2026-08-18T03:14:14.459Z
Formal review: #3452 (review)

The formal GitHub review is the authoritative result. This comment only reports operational status.

@avocet-bot avocet-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review model: Claude Opus

Model verdict: APPROVE

No blocking findings. The current head is a merge of main into the branch; the exact base/head diff is identical to the previously reviewed head (same 30 files, same edits), and the only delta between the two heads is examples/tutorials/sveltekit.md, which came from main and is identical in base and head (not part of this changeset). All prior verification still holds: the eight changed docs.deno.com links resolve to existing files/anchors in this repo (which serves docs.deno.com) — runtime/getting_started/{installation,setup_your_environment}.md, deploy/reference/cron.md, deploy/kv/index.md (old /deploy/kv/manual is now its oldUrl), deploy/classic/queues.md (fixes class9cclassic), deploy/classic/api/runtime-broadcast-channel.md, runtime/reference/cli/bench.md, and the #use-a-node-built-in-module anchor in node.md. Internal /examples/ links now use the underscore convention (writing_files, http_server_routing) and module_metadata_tutorial (confirmed via tutorial frontmatter and _data.ts); the removed hyphenated dependency-management link targeted a nonexistent example. Prose fixes are all correct. No code, tests, error handling, or types are affected.

@magurotuna
magurotuna force-pushed the fix/example/links-prose branch 2 times, most recently from c1ba57b to f19f097 Compare August 18, 2026 03:46
@magurotuna
magurotuna force-pushed the fix/example/links-prose branch from f19f097 to 72c5ea5 Compare August 18, 2026 03:56
@magurotuna
magurotuna merged commit d1fce0f into main Aug 18, 2026
3 checks passed
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