Skip to content

feat(posts): add EP75 TypeScript 7 Go rewrite episode - #86

Merged
AnnatarHe merged 2 commits into
mainfrom
feat/ep75-typescript7-go
Sep 15, 2026
Merged

AnnatarHe merged 2 commits into
mainfrom
feat/ep75-typescript7-go

Conversation

@AnnatarHe

Copy link
Copy Markdown
Member

Adds EP75, “TypeScript 7:十倍性能的升级之外,为什么选 Go?”, at /posts/ep75/, with the supplied shownotes, 12 categories, portrait cover, and downloadable SRT transcript. Publication date is September 14, 2026; xyzLink and biliUrl remain empty placeholders for later updates.

Validation:

  • env npm_config_manage_package_manager_versions=false pnpm astro check — passed, with two existing unused-symbol hints in the thumbnail script.
  • env npm_config_manage_package_manager_versions=false pnpm build — passed, including SEO validation for 80 HTML pages, 80 sitemap URLs, and 78 RSS items.
  • Browser inspection confirmed the cover layout, article text, YouTube entry, and transcript expansion (34 paragraphs). Verified the subtitle HTTP download matches the supplied file, EP75 appears in the posts listing and RSS, RSS metadata includes its date/summary/cover, and generated Open Graph text fits.

Checks used installed pnpm 12.0.0 because automatic activation of pinned pnpm 12.1.0 could not verify its registry identity due to registry request failures; project configuration is unchanged. The SRT is copied byte-for-byte, including its original trailing whitespace.

The supplied YouTube video (OMIfl3o8Z0g) currently reports “This video is private” in the embedded player, so public playback remains unavailable. Its ID is retained as supplied.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-14T16:07:55.820662Z 98eca88 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 98eca88624

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

xyzLink: ""
youtubeId: OMIfl3o8Z0g
biliUrl: ""
cover: ../../assets/ep75/cover.png

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Replace the portrait asset with a square cover

The referenced PNG is 1086×1448 rather than square, so EpisodeCover.astro renders EP75 as a tall 3:4 image via h-auto, disrupting the intended episode-header layout and violating the repository's square-cover convention. Crop or replace it with a square asset before publishing.

AGENTS.md reference: AGENTS.md:L3-L5

Useful? React with 👍 / 👎.

@AnnatarHe
AnnatarHe merged commit 5127ca1 into main Sep 15, 2026
1 check failed
@AnnatarHe
AnnatarHe deleted the feat/ep75-typescript7-go branch September 15, 2026 12:55

This branch was successfully deployed

No deployments
feat/ep75-typescript7-go - asynctalk-website PR #86 e63a919d Deployed Sep 15, 2026 by AnnatarHe
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