-
Notifications
You must be signed in to change notification settings - Fork 49
chore: replace my site reference with main site #253
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: main
Are you sure you want to change the base?
Conversation
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings. WalkthroughDocumentation update to redirect URL in authentication callback sections, changing the target from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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
🤖 Fix all issues with AI agents
In @auth/README.md:
- Line 72: The README contains bare URLs (e.g., the redirect URL
"https://www.realdevsquad.com/new-signup" referenced in the sentence "Redirects
to the specified redirectURL or https://www.realdevsquad.com/new-signup if user
details are incomplete." and the other instance at line 163) which violate
MD034; update those occurrences by wrapping the URLs in angle brackets (e.g.,
<https://www.realdevsquad.com/new-signup>) so the markdown is lint-compliant.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
auth/README.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
auth/README.md
72-72: Bare URL used
(MD034, no-bare-urls)
163-163: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
auth/README.md (1)
70-77: Documentation update correctly reflects the redirect URL change.The changes to both the Google and GitHub callback endpoint documentation are consistent and accurately document the new fallback redirect URL (
https://www.realdevsquad.com/new-signup) when user details are incomplete, replacing the previousmy.realdevsquad.comreference. The documentation changes align with the PR objective.Also applies to: 161-168
| - **Success Response**: | ||
| - **Code**: 302 | ||
| - **Content**: Redirects to the specified redirectURL or https://my.realdevsquad.com/new-signup if user details are incomplete. | ||
| - **Content**: Redirects to the specified redirectURL or https://www.realdevsquad.com/new-signup if user details are incomplete. |
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.
Wrap bare URLs in angle brackets for markdown compliance.
Lines 72 and 163 contain bare URLs that should be wrapped in angle brackets (<url>) to comply with the MD034 markdown linting rule. This ensures consistent formatting across the documentation.
📝 Proposed fix
- - **Content**: Redirects to the specified redirectURL or https://www.realdevsquad.com/new-signup if user details are incomplete.
+ - **Content**: Redirects to the specified redirectURL or <https://www.realdevsquad.com/new-signup> if user details are incomplete.Also applies to: 163-163
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
72-72: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In @auth/README.md at line 72, The README contains bare URLs (e.g., the redirect
URL "https://www.realdevsquad.com/new-signup" referenced in the sentence
"Redirects to the specified redirectURL or
https://www.realdevsquad.com/new-signup if user details are incomplete." and the
other instance at line 163) which violate MD034; update those occurrences by
wrapping the URLs in angle brackets (e.g.,
<https://www.realdevsquad.com/new-signup>) so the markdown is lint-compliant.
Date: 10-01-26
Developer Name: @MayankBansal12
Issue Ticket Number
Description
Documentation Updated?
Under Feature Flag
Breaking Changes
Screenshots
N/A