Development

[assembler-0]

Summary by CodeRabbit

• Chores

  • Enforced a C++20-or-later requirement with a compile-time guard to prevent incompatible builds.

• Documentation

  • Updated examples to reflect the current result access pattern.
  • Clarified error reporting text and improved wording.
  • Fixed typos and minor editorial issues in code samples and prose.

• Style

  • Minor formatting cleanup in a sample/utility source file (no logic changes).

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Adds a C++20 compile-time gate around CatalystCX.hpp, preventing inclusion on older standards. Makes minor formatting change in Evaluate.cpp. Updates README.md examples to access results via Value(), fixes typos, and adjusts sample variable bindings.

Changes

Cohort / File(s)	Summary of changes
Build standard gate CatalystCX.hpp	Adds a compile-time check for C++20 (__cplusplus >= 202002L); emits error and defines CATALYST_VERSION_CHECK_FAILED otherwise. Gates the remainder of the header on this macro. No runtime/API changes.
Docs usage updates README.md	Updates code samples to use Value() before accessing fields (e.g., result.Value().Stdout), switches some bindings to auto&, fixes typos (e.g., Vaule→Value), and minor punctuation edit.
Formatting only Evaluate.cpp	Inserts an extra blank line; no logic or control-flow changes.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Development #2 — Also modifies CatalystCX.hpp with API/type additions; impacted by or complementary to the new C++20 inclusion gate.
• Development #6 — Changes in CatalystCX.hpp that would be conditionally compiled; directly related to the new compile-time guard.

Poem

A bunny taps the build with care,
“C++20 only—mind the air!”
Docs now hop through Value’s door,
Typos fixed, examples soar.
A whisker twitch, a tidy line—
Compile-time gates, and all is fine. 🐇✨

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch Development

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 639356d and 1214636.

📒 Files selected for processing (3)

• CatalystCX.hpp (2 hunks)
• Evaluate.cpp (1 hunks)
• README.md (8 hunks)

---

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.}