-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
fix: absolute path issue in link for consistent URLs #3856
fix: absolute path issue in link for consistent URLs #3856
Conversation
Fixed absolute path issue in link for consistent URLs
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.
You don't have to update the versioned docs. Please remove the change and this PR will be good to go. Thanks 😄
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.
Thanks for the review, revered the changes
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3856 +/- ##
=======================================
Coverage 79.69% 79.69%
=======================================
Files 87 87
Lines 1842 1842
Branches 818 818
=======================================
Hits 1468 1468
Misses 369 369
Partials 5 5 ☔ View full report in Codecov by Sentry. |
Please update the branch with the base branch so that I can merge the PR. |
Done |
Motivation
Fixes #3855
Type of change
How Has This Been Tested?
example
appChecklist
Additional context