Skip to content

docs: lean CLAUDE.md (claude-md-lint)#19

Merged
saagpatel merged 1 commit into
mainfrom
docs/claude-md-lint
May 31, 2026
Merged

docs: lean CLAUDE.md (claude-md-lint)#19
saagpatel merged 1 commit into
mainfrom
docs/claude-md-lint

Conversation

@saagpatel
Copy link
Copy Markdown
Owner

Leaned the project CLAUDE.md via the claude-md-lint skill: positive-framing over prohibition walls, trimmed stale status narration, with a deterministic no-information-loss gate preserving every build/test/run command, URL, path, port, and env var. The auto-generated portfolio-context block is untouched.

@saagpatel saagpatel merged commit fe97dbb into main May 31, 2026
@saagpatel saagpatel deleted the docs/claude-md-lint branch May 31, 2026 07:19
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3986819a2b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread CLAUDE.md
Comment on lines +36 to +37
cargo clippy -- -D warnings # lint (must pass clean)
cargo fmt # format on save
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Point Rust tooling at src-tauri manifest

In the documented repo-root workflow, these Rust commands fail because this repository has no root Cargo.toml; the only manifest is src-tauri/Cargo.toml, and running cargo clippy -- -D warnings from /workspace/ModelColosseum exits with could not find Cargo.toml. Cargo's own help lists --manifest-path <PATH> for clippy, so this block should either cd src-tauri or pass --manifest-path src-tauri/Cargo.toml for the Rust lint/format steps, otherwise agents following CLAUDE.md cannot run the stated clean gate.

Useful? React with 👍 / 👎.

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