Skip to content

docs(logger): fix "destinatin" typo in JSDoc#16818

Draft
adv0r wants to merge 1 commit into
withastro:mainfrom
adv0r:tokenburn/fix-destinatin-jsdoc-typo
Draft

docs(logger): fix "destinatin" typo in JSDoc#16818
adv0r wants to merge 1 commit into
withastro:mainfrom
adv0r:tokenburn/fix-destinatin-jsdoc-typo

Conversation

@adv0r
Copy link
Copy Markdown

@adv0r adv0r commented May 21, 2026

Dear maintainer — this PR has a permanent home with methodology + opt-out at tokens-for-good. A one-line "no thanks" → auto-close + blacklist. Sorry for the notification this edit caused.


Heads-up: I realise this PR may not be welcome here. It was prepared
by an AI agent (Cursor / Claude Opus 4.7) running unattended under
@adv0r as a small personal token-burn initiative. I deliberately
picked a low-impact, easy-to-review, narrowly-scoped change so a
maintainer can either land it in a minute or close it with zero
guilt. No hard feelings either way — just trying to be useful while
spending some leftover Cursor credits before the billing cycle ends.

What

Fixes a one-character typo in a JSDoc comment in packages/astro/src/core/logger/core.ts:

-	 * It calls the `flush` function of the provided destinatin, if it exists.
+	 * It calls the `flush` function of the provided destination, if it exists.

Why

Trivial doc copy-edit. The rest of the file consistently uses destination (it's also the actual property name: this.options.destination), so the typo is clearly unintentional.

How verified

Pure doc-comment change — no runtime / type / public-API impact. JSDoc only, no test fixtures affected.

Maintainer note: feel free to close

If a one-character JSDoc typo isn't worth the round-trip overhead for you, please just close — zero hard feelings.

Co-authored-by: Cursor <cursoragent@cursor.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 21, 2026

⚠️ No Changeset found

Latest commit: e47be97

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions Bot added the pkg: astro Related to the core `astro` package (scope) label May 21, 2026
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 21, 2026

Merging this PR will not alter performance

✅ 18 untouched benchmarks


Comparing adv0r:tokenburn/fix-destinatin-jsdoc-typo (e47be97) with main (1c8dcc8)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant