-
Notifications
You must be signed in to change notification settings - Fork 967
Fix wrong links to "Heterogeneous Lookup in Associative Containers" #5396
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix wrong links to "Heterogeneous Lookup in Associative Containers" #5396
Conversation
@Rageking8 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
Learn Build status updates of commit fd729a9: ✅ Validation status: passed
For more details, please refer to the build report. |
@TylerMSFT - Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
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.
@Rageking8 - I have a question about the removal of -c14. Looks like it is necessary. This is what I see when I copy the anchor link in the browser:
#heterogeneous-lookup-in-associative-containers-c14 <--- c14
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.
Thank you for sending folks to the right place.
#sign-off |
Not sure why so many of these links point to the wrong
sequence_containers
anchor. Theheterogeneous-lookup-in-associative-containers
anchor is added as the defaultheterogeneous-lookup-in-associative-containers-c14
one might give wrong connotations about C (although C14 does not actually exist). Kebab case is chosen over snake case since the default one is as such.