Skip to content

Add claude.md and fix smoke test.#283

Merged
leo-lem merged 3 commits into
mainfrom
claude+smoke
Jul 8, 2026
Merged

Add claude.md and fix smoke test.#283
leo-lem merged 3 commits into
mainfrom
claude+smoke

Conversation

@leo-lem

@leo-lem leo-lem commented Jul 8, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 8, 2026 07:29
@leo-lem leo-lem enabled auto-merge (squash) July 8, 2026 07:29

Copilot AI 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.

Pull request overview

This PR adds repository-specific guidance for Claude Code via CLAUDE.md, and updates the Playwright smoke test to reflect how Astro redirects behave when deployed on GitHub Pages (static hosting).

Changes:

  • Added CLAUDE.md with repo architecture notes and common dev/test/tooling commands.
  • Updated smoke test coverage for “removed pages” to validate GitHub Pages’ meta-refresh redirect behavior instead of expecting an HTTP 301.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
test/smoke.spec.ts Adjusts redirect smoke test to use browser navigation and assert homepage arrival/title under static-host redirect behavior.
CLAUDE.md Adds a Claude Code guidance document describing repo structure, commands, content sync, and CI behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/smoke.spec.ts Outdated
Comment thread CLAUDE.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

@leo-lem leo-lem self-assigned this Jul 8, 2026
@leo-lem leo-lem merged commit 46a82f1 into main Jul 8, 2026
3 checks passed
@leo-lem leo-lem deleted the claude+smoke branch July 8, 2026 07:37
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