Skip to content

feat: update author weekly update - #35

Merged
dewabisma merged 1 commit into
mainfrom
beast/update-author-weekly-update
Aug 26, 2026
Merged

feat: update author weekly update#35
dewabisma merged 1 commit into
mainfrom
beast/update-author-weekly-update

Conversation

@dewabisma

Copy link
Copy Markdown
Collaborator

Summary

Properly attribute author in weekly update.

@dewabisma
dewabisma requested a review from n13 August 26, 2026 06:11

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

Reviewer model: GPT Sol

APPROVE — Weekly-update entries now resolve to Christopher Smith unless an explicit author is set, and the resolved identity is consistently used for the visible byline, Article JSON-LD, and Twitter creator metadata.

No blocking findings.

Validation:

  • Reviewed the complete 704c1ad...e620bb3 diff and affected collection/rendering paths.
  • git diff --check passed.
  • Frozen Bun dependency installation passed with browser download disabled for the local environment.
  • bun run check:seo-meta passed, and 6 focused resolver cases passed.
  • Astro generated every localized route and optimized assets; rendered weekly pages contain Christopher Smith, /team/christopher-smith, and twitter:creator=@YuviLightman. Compression completed for 1,406 HTML files, although the local command was stopped after the integration did not exit promptly.
  • GitHub Actions reported startup_failure before creating any jobs or logs, so no remote checks ran.

Non-blocking follow-up: src/content.config.ts is the only changed file rejected by Prettier (its base version passes); running Prettier on that file would keep the patch consistent. The repository-wide format check also reports 15 unrelated pre-existing files.

@dewabisma
dewabisma merged commit c923a04 into main Aug 26, 2026
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