-
Notifications
You must be signed in to change notification settings - Fork 782
Clean up "here" link anchors for docs, ee, and frontend dirs #29742
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #29742 +/- ##
=======================================
Coverage 64.23% 64.23%
=======================================
Files 1852 1852
Lines 180476 180501 +25
Branches 5248 5255 +7
=======================================
+ Hits 115926 115944 +18
- Misses 55436 55443 +7
Partials 9114 9114
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Also includes a new subheading for human-device mapping in profiles article
9539924 to
788d5a7
Compare
|
Rebased this to be independent of #29731. Will add a 3rd PR for performing the same fixes on articles. |
|
@rachaelshaw @mostlikelee @georgekarrv I think I need approvals from all three of y'all to merge this. |
|
@rachaelshaw changes look good to me! Please feel free to give your CODEOWNER approval. (FYI I'm reviewing for Rachael to give Rachael some time back for drafting.) |
|
@lukeheath Need your +1 once more here due to changes made since your review. Looks like we're good to go at that point. Thx! |
lukeheath
left a 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.
🙌
More work to fix #29720.