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
[iterators] Add missing definitions from the iterators library to the library index #6011
[iterators] Add missing definitions from the iterators library to the library index #6011
Conversation
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.
Using \libglobal
instead of \indexlibraryglobal
should index to the correct page, and raise the confidence by inspection that we're not indexing into a definition that says see below
.
Hi @JohelEGP , thanks a lot for your feedback. So is it |
That's right.
I'd have to manually check if you're right. But if you used |
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.
Please, check out the file to see how \libglobal
is used and fix the PR.
Obviously this is my first contribution. I am very thankful for your helpful and especially very quick support, @JohelEGP . Thank you very much! |
You can fixup the commits into a single one formatted as described at https://github.com/cplusplus/draft/wiki/Commit-message-format#editorial-commits. |
977dd14
to
6a5dc66
Compare
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.
Please fix the merge conflicts.
6a5dc66
to
0d96935
Compare
@jensmaurer Can you reapprove? |
@tkoeppe, I can. |
Thanks, Jens, esp. for avoiding undefined modal verbs. @Ukilele Thank you very much for your contribution! |
Add following definitions from the iterators library to the library index: