-
Notifications
You must be signed in to change notification settings - Fork 31
docs: fix typo #266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
docs: fix typo #266
Conversation
✅ Deploy Preview for confidential-tokens ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughChanged spelling from British "licence" to American "license" in the README's License section and adjusted end-of-file formatting metadata without altering visible content. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
README.md
24-24: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
README.md (1)
20-20: Typo fix looks good.The spelling correction from "licence" to "license" aligns with American English conventions and improves consistency in the documentation.
| ## Legal | ||
|
|
||
| Your use of this Project is governed by the terms found at www.openzeppelin.com/tos (the "Terms"). | ||
| Your use of this Project is governed by the terms found at www.openzeppelin.com/tos (the "Terms"). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bare URL should be wrapped in markdown link syntax.
Markdown linters expect bare URLs to be wrapped in angle brackets or proper link syntax for better accessibility and consistency.
Apply this diff to wrap the bare URL:
-Your use of this Project is governed by the terms found at www.openzeppelin.com/tos (the "Terms").
+Your use of this Project is governed by the terms found at <www.openzeppelin.com/tos> (the "Terms").📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Your use of this Project is governed by the terms found at www.openzeppelin.com/tos (the "Terms"). | |
| Your use of this Project is governed by the terms found at <www.openzeppelin.com/tos> (the "Terms"). |
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
24-24: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In README.md around line 24, the bare URL "www.openzeppelin.com/tos" should be
wrapped in proper Markdown link syntax; replace the bare URL with either an
inline link like [Terms of Service](https://www.openzeppelin.com/tos) or
angle-bracketed URL <https://www.openzeppelin.com/tos> (prefer the descriptive
link text) so linters and accessibility tools recognize it correctly.
Hello! Fixed minor typo licence -> license
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.