Skip to content
This repository was archived by the owner on Jun 3, 2026. It is now read-only.

Remove leftover files from mkdocs site:#692

Open
zastrowm wants to merge 1 commit into
strands-agents:mainfrom
zastrowm:remove_old_leftovers
Open

Remove leftover files from mkdocs site:#692
zastrowm wants to merge 1 commit into
strands-agents:mainfrom
zastrowm:remove_old_leftovers

Conversation

@zastrowm

Copy link
Copy Markdown
Member

Description

Removes the legacy MkDocs configuration (mkdocs.yml, extra.css, and the monolithic quickstart.mdx) that were left over from the MkDocs-to-Astro migration. The quickstart content has already been split into the src/content/docs/user-guide/quickstart/ directory, so the old single-page file was dead weight.

Also adds a redirect from /docs/user-guide/quickstart to the new overview page so existing links and bookmarks don't 404, and registers that route in the known-routes test fixture.

Related Issues

Type of Change

  • Structure/organization improvement
  • Bug fix

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using npm run dev
  • Links in the documentation are valid and working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

 - quickstart.mdx -> our overview handles that
 - a bunch of other random files
@github-actions

Copy link
Copy Markdown
Contributor

Assessment: Request Changes

Clean PR removing legacy MkDocs artifacts from the migration to Astro. The redirect and test fixture updates are correctly implemented.

Review Details
  • Consistency: AGENTS.md needs updating to remove the mkdocs.yml reference from the directory structure, since that file is being deleted by this PR.
  • Verified: Redirect from /docs/user-guide/quickstart is properly configured via redirectFrom in overview.mdx.
  • Verified: known-routes.json test fixture correctly includes the new route.

One small update needed to keep the documentation in sync with the codebase.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation Preview Failed

The documentation deployment encountered an error. Please check the deployment logs for more details.

@zastrowm zastrowm mentioned this pull request May 22, 2026
5 tasks
@strands-agent

Copy link
Copy Markdown
Contributor

This repository has been merged into the strands-agents/harness-sdk monorepo and will be archived shortly. All new development happens there.

If this PR is still relevant, please recreate it against the monorepo. The code now lives under site/. Full commit history was preserved, so your base should be findable.

Apologies for the disruption, and thank you for contributing!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants