Skip to content

Remove leftover files from mkdocs site:#692

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

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

Conversation

@zastrowm
Copy link
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
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
Contributor

Documentation Preview Failed

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

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