Skip to content

Fix typos in documentation - #499

Merged
wchilders-nvidia merged 1 commit into
edgcpp:mainfrom
hamidzr:fixtypos
Nov 9, 2026
Merged

wchilders-nvidia merged 1 commit into
edgcpp:mainfrom
hamidzr:fixtypos

Conversation

@hamidzr

@hamidzr hamidzr commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Documentation
    • Corrected spelling errors in the contribution guidelines and native build instructions.

@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

This change corrects spelling errors in contribution guidance and Linux and MacOS native build instructions. It does not change the documented guidance or build behavior.

Changes: Documentation spelling corrections

Layer / File(s) Summary
Correct spelling in guidance
CONTRIBUTING.md, HACKING.md
Corrected spelling in contribution guidance and Linux and MacOS build instructions.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other · Severity of issue fixed: Low

Suggested reviewers: darkarc

Merge Risk: Minimal · up to b49e0

This PR fixes typos in documentation and does not change behavior. It is safe to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "Fix typos in documentation" matches the changeset. The raw summary shows spelling corrections in CONTRIBUTING.md (feature-request instructions, AI-assisted contribution guidance) and HACKIN…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "Fix typos in documentation" matches the changeset. The raw summary shows spelling corrections in CONTRIBUTING.md (feature-request instructions, AI-assisted contribution guidance) and HACKING.md (the word "procedure" in the Linux and macOS native build instructions). The title names the main change and is specific enough for a teammate scanning history.

Finishing Touches

Generate unit tests (beta)

  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 14e7792 and b49e0b5.

Files selected for processing: 2

  • CONTRIBUTING.md
  • HACKING.md
Files skipped from review as they are similar to previous changes: 2

  • HACKING.md
  • CONTRIBUTING.md

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

@edgcpp-bot

Copy link
Copy Markdown
Collaborator

AcknowlEDG review results updated for 14e7792fced7:

CI success.

Signed-off-by: Hamid Zare <12127420+hamidzr@users.noreply.github.com>

@wchilders-nvidia wchilders-nvidia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! These are admittedly all my fault, I wrote those 😬 😅

@edgcpp-bot

Copy link
Copy Markdown
Collaborator

AcknowlEDG review results updated for b49e0b5914fa:

CI success.

@wchilders-nvidia
wchilders-nvidia added this pull request to the merge queue Nov 9, 2026
Merged via the queue into edgcpp:main with commit 73b1b0f Nov 9, 2026
21 checks passed
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.

3 participants