Skip to content

chore: mark sponsor links as sponsored and shorten past-sponsors list - #58

Merged
dinwwwh merged 4 commits into
mainfrom
claude/sync-sponsors-standard-06e8eb
Aug 10, 2026
Merged

chore: mark sponsor links as sponsored and shorten past-sponsors list#58
dinwwwh merged 4 commits into
mainfrom
claude/sync-sponsors-standard-06e8eb

Conversation

@dinwwwh

@dinwwwh dinwwwh commented Aug 10, 2026

Copy link
Copy Markdown
Member

Two changes to the generated README sponsors section.

Sponsor links are paid placements, so they now declare that with the standard sponsored link type: rel="noopener" becomes rel="sponsored noopener".

The past-sponsors block rendered 38 avatar images, which visually dwarfed the active tiers above it. It is now a single sentence — "With thanks to 38 past sponsors who helped get us here." — with the ### Past Sponsors heading dropped.

Changes

  • scripts/sync-sponsors.ts emits both the new rel value and the summary line (with singular/plural handling for the count).
  • The 10 checked-in README.md files were regenerated, removing ~40 lines of avatar markup each.

Testing

Regenerated via node scripts/sync-sponsors.ts against live sponsor data; the diff touches only the rel attribute and the past-sponsors block, with no unrelated sponsor churn. Lint-staged eslint passed on both commits.

Sponsor links in the generated README sponsors section are paid
placements, so they now carry the standard `sponsored` link type
alongside `noopener` for the `target="_blank"` destination.
@pkg-pr-new

pkg-pr-new Bot commented Aug 10, 2026

Copy link
Copy Markdown
@standardserver/aws-lambda

npm i https://pkg.pr.new/@standardserver/aws-lambda@58

@standardserver/bun

npm i https://pkg.pr.new/@standardserver/bun@58

@standardserver/core

npm i https://pkg.pr.new/@standardserver/core@58

@standardserver/deno

npm i https://pkg.pr.new/@standardserver/deno@58

@standardserver/fastify

npm i https://pkg.pr.new/@standardserver/fastify@58

@standardserver/fetch

npm i https://pkg.pr.new/@standardserver/fetch@58

@standardserver/node

npm i https://pkg.pr.new/@standardserver/node@58

@standardserver/peer

npm i https://pkg.pr.new/@standardserver/peer@58

@standardserver/shared

npm i https://pkg.pr.new/@standardserver/shared@58

commit: d5e77d7

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@pullfrog pullfrog 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.

✅ No new issues found.

Reviewed changes — Reviewed the single commit 50d0ba8 across 11 files (a script change plus 10 regenerated READMEs).

  • scripts/sync-sponsors.ts — both sponsor-link templates (tier tables and past-sponsors list) now emit rel="sponsored noopener" instead of rel="noopener".
  • 10 checked-in README.md files — the rel attribute on every sponsor link updated to rel="sponsored noopener", no other sponsor data changed.

The change is consistent and correct: the script's two emission sites match the README output, and a repo-wide grep confirms no bare rel="noopener" remains in any sponsor link. This is a pure markup/SEO metadata change with no runtime behavioral surface.

Pullfrog  | View workflow run | Using DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

@codspeed-hq

codspeed-hq Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 108 skipped benchmarks1


Comparing claude/sync-sponsors-standard-06e8eb (d5e77d7) with main (bd63043)

Open in CodSpeed

Footnotes

  1. 108 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

The past-sponsors list rendered 38 avatar images, which dwarfed the
active tiers above it. It is now one sentence crediting the count.
@dinwwwh dinwwwh changed the title chore: mark sponsor links with rel="sponsored" chore: mark sponsor links as sponsored and shorten past-sponsors list Aug 10, 2026

@pullfrog pullfrog 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.

✅ No new issues found.

Reviewed changes — Reviewed the incremental delta commit 8d458b2 (summarize past sponsors as a single line) on top of the previously-approved 50d0ba8.

  • Replaced the past-sponsors avatar block in buildSponsorsSection with a single thank-you sentence (With thanks to ${count} ${noun} who helped get us here., singular/plural aware) — the ### Past Sponsors heading, <p>, and 38-avatar loop are removed from the generator.
  • Regenerated all 10 checked-in README.md files; every sponsors section now ends at </table> followed by the summary sentence.

Repo-wide grep confirms the state is consistent: zero bare rel="noopener", zero residual ### Past Sponsors heading, no stray avatar <p> markup, and all 10 READMEs carrying the With thanks to … line — matching the generator output exactly. Pure README/generator change with no runtime behavioral surface.

Pullfrog  | View workflow run | Using DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

@pullfrog pullfrog 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.

✅ No new issues found.

Reviewed changes — Reviewed the incremental delta commit def0981 (link the past-sponsors line to GitHub Sponsors) on top of the previously-approved 8d458b2.

  • scripts/sync-sponsors.ts — the past-sponsors thank-you sentence is now emitted as a markdown link to https://github.com/sponsors/dinwwwh.
  • Regenerated all 10 checked-in README.md files, each carrying the identical linked line.

Repo-wide verification confirms the generator and all READMEs stay in sync. Pure README/generator markup change with no runtime behavioral surface.

Pullfrog  | View workflow run | Using DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

@dinwwwh
dinwwwh merged commit 6cc7812 into main Aug 10, 2026
10 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.

1 participant