Skip to content

Fix stale path references in WARP.md#9336

Open
anshul-garg27 wants to merge 2 commits intowarpdotdev:masterfrom
anshul-garg27:fix/warp-md-paths
Open

Fix stale path references in WARP.md#9336
anshul-garg27 wants to merge 2 commits intowarpdotdev:masterfrom
anshul-garg27:fix/warp-md-paths

Conversation

@anshul-garg27
Copy link
Copy Markdown

Description

A few path references in WARP.md don't match the current workspace layout. The "Core Libraries" bullets list warp_core/, editor/, ui/, ipc/, and graphql/ as if they live at the repo root, but each one is under crates/. The "Workspace Structure" line still says "34+ member crates" — find crates -maxdepth 1 -mindepth 1 -type d returns 63 today. And the Database / GraphQL sections point at paths that don't resolve:

  • migrations/crates/persistence/migrations/
  • app/src/persistence/schema.rscrates/persistence/src/schema.rs
  • graphql/api/schema.graphqlcrates/warp_graphql_schema/api/schema.graphql

Updated each reference to match what's on disk.

Testing

Docs-only change.

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @anshul-garg27 on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment @cla-bot check to trigger another check.

@oz-for-oss
Copy link
Copy Markdown

oz-for-oss Bot commented Apr 28, 2026

@anshul-garg27

I'm starting a first review of this pull request.

You can follow along in the session on Warp.

I approved this pull request and requested human review from: @vorporeal, @alokedesai, @zachbai.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@cla-bot cla-bot Bot added the cla-signed label Apr 28, 2026
Copy link
Copy Markdown

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This docs-only PR updates stale WARP.md path references to match the current repository layout and adjusts the workspace crate count wording.

Concerns

  • None. The updated paths exist in the checkout, and no security-relevant code or configuration changes were introduced.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@anshul-garg27
Copy link
Copy Markdown
Author

@cla-bot check

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

The cla-bot has been summoned, and re-checked this pull request!

Copy link
Copy Markdown
Member

@szgupta szgupta left a comment

Choose a reason for hiding this comment

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

ty!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants