Skip to content

documentation: fix typo and phrasing#2298

Merged
glennjacobs merged 2 commits intolunarphp:1.xfrom
jonagoldman:1.x
Sep 23, 2025
Merged

documentation: fix typo and phrasing#2298
glennjacobs merged 2 commits intolunarphp:1.xfrom
jonagoldman:1.x

Conversation

@jonagoldman
Copy link
Contributor

@jonagoldman jonagoldman commented Sep 23, 2025 •
edited by coderabbitai bot
Loading

Just a typo fix in the docs.

Summary by CodeRabbit

  • Documentation
    • Fixed typographical errors in the Orders reference (replaced “you’re own” with “your own” in two places related to cart/validator guidance).
    • Reworded a section heading for clarity: "Order Reference Generating" → "Order Reference Generation".
    • Minor content alignment edits to improve readability; no behavioral or functional changes.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 23, 2025 •
edited
Loading

Walkthrough

Fixed typographical and heading wording in documentation: corrected "you're own" to "your own" in two places and changed "Order Reference Generating" to "Order Reference Generation" in docs/core/reference/orders.md. No code or behavioral changes.

Changes

Cohort / File(s) Summary of Changes
Docs
docs/core/reference/orders.md		 Corrected grammar: replaced "you're own" with "your own" in two descriptions and reworded heading from "Order Reference Generating" to "Order Reference Generation". No functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "documentation: fix typo and phrasing" accurately and concisely describes the PR's primary change of correcting typos and minor wording in documentation; it is relevant to the provided changeset and clear for a reviewer scanning history. It is neither misleading nor overly vague and does not include unnecessary noise.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 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 3a2114e and cd4346d.

📒 Files selected for processing (1)
  • docs/core/reference/orders.md (2 hunks)
🔇 Additional comments (2)
docs/core/reference/orders.md (2)

133-133: Optional: add backward-compatible anchor for renamed heading

Repo search found no internal references to order-reference-generating; add the anchor only if you want to preserve external/backlinks.

Location: docs/core/reference/orders.md:133

Proposed change:

-## Order Reference Generation
+<a id="order-reference-generating"></a>
+## Order Reference Generation

100-100: Typo fix LGTM. Searched the repo for "you're own" variants (ASCII and unicode apostrophes) — no other occurrences found.

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

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

coderabbitai[bot]
coderabbitai bot reviewed Sep 23, 2025
View reviewed changes
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/core/reference/orders.md (1)

133-135: Optional: Improve heading phrasing.

Consider "Order Reference Generation" for clarity.

-## Order Reference Generating
+## Order Reference Generation
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7586c34 and 3a2114e.

📒 Files selected for processing (1)
  • docs/core/reference/orders.md (1 hunks)
🔇 Additional comments (1)
docs/core/reference/orders.md (1)

100-101: Typo fixed correctly.

"your own" is correct here. Thanks for tightening the docs.

@jonagoldman jonagoldman changed the title documentation: fix a typo documentation: fix typo and phrasing Sep 23, 2025
@glennjacobs glennjacobs merged commit b64b59c into lunarphp:1.x Sep 23, 2025
1 check passed
@vercel
Copy link

vercel bot commented Sep 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
lunar-docs Ready Ready Preview Comment Sep 23, 2025 4:02pm

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

Reviewers

@glennjacobs glennjacobs glennjacobs approved these changes

+1 more reviewer

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jonagoldman @glennjacobs