Update Claude.md

[hardikdr]

Summary by CodeRabbit

• Documentation
  • Expanded contributor guidelines with operational standards and change discipline requirements.
  • Added end-to-end verification checklist with specific build and testing commands for development workflows.

[coderabbitai]

Walkthrough

Updated CLAUDE.md with new contribution guidance including Git/GitHub operational restrictions, scoped change discipline requirements, status-conditions patching patterns for ServerBootConfiguration, expanded documentation and test expectations, and a mandatory end-to-end verification checklist. No functional code changes.

Changes

Cohort / File(s)	Summary
Documentation & Contribution Guidance CLAUDE.md	Added sections defining Git/GitHub operational prohibitions, controller change discipline, required status-conditions patching pattern with retry.RetryOnConflict and client.MergeFrom, documentation/test update expectations, and mandatory e2e verification checklist with specific make commands.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• ironcore-dev/boot-operator#296 — Overlapping documentation changes to CLAUDE.md

Suggested labels

chore, area/metal-automation, size/L

Suggested reviewers

• defo89

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is empty with no proposed changes, affected files, or issue references provided.	Add a meaningful description following the template: list proposed changes, identify affected files, and reference any related issues using 'Fixes #' if applicable.
Title check	❓ Inconclusive	The title 'Update Claude.md' is vague and generic—it does not specifically convey what was updated or why, making it unclear to someone scanning commit history.	Use a more descriptive title that specifies the main additions, such as 'Add contribution rules and verification checklist to CLAUDE.md' or 'Update CLAUDE.md with Git operational guidelines and test conventions'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/update-claude-md-verification-rules

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

CLAUDE.md (1)

141-141: Minor wording polish for concision.

Consider “run all the following commands” instead of “run all of the following commands.”

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@CLAUDE.md` at line 141, Replace the phrase "run all of the following
commands" to remove the extra word "of" in the sentence; update the line that
currently reads "run all of the following commands and fix any failures before
presenting the result for review." to the more concise "run all the following
commands and fix any failures before presenting the result for review." in the
CLAUDE.md text.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@CLAUDE.md`:
- Line 141: Replace the phrase "run all of the following commands" to remove the
extra word "of" in the sentence; update the line that currently reads "run all
of the following commands and fix any failures before presenting the result for
review." to the more concise "run all the following commands and fix any
failures before presenting the result for review." in the CLAUDE.md text.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4e2a9fe8-c19c-46ef-b210-3d78ea1457f9

📥 Commits

Reviewing files that changed from the base of the PR and between 7f1e41a and 5605716.

📒 Files selected for processing (1)

• CLAUDE.md