-
Notifications
You must be signed in to change notification settings - Fork 444
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
Docs: Initial sequence of pages in Documentation #4758
Conversation
e12997b
to
c152a41
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.
LGTM
@fruffy It appears DCO is required in order to click the Merge button on Github. Note: That sounds GOOD to me. Just wanted to let you know, since you mentioned recently that you thought it was optional right now. @AdarshRawat1 I think the only thing needed to commit this is for you to add DCO. You can can click the "Details" link to the right of the failing check labeled "DCO" for instructions on how. |
I think this is because this is a draft pull request. |
fc1db64
to
9c7e147
Compare
Thanks, Done !! |
I was wrong. You also need to change the PR so it is no longer in Draft status. |
I'm just testing to see if anything breaks or is not being rendered. [The list of classes and namespaces]. |
Seems like this PR has broken CI, I would use rebase merge instead of conventional merging. It works much better on Github. |
Signed-off-by: Adarsh <[email protected]>
Signed-off-by: Adarsh <[email protected]>
Signed-off-by: Adarsh <[email protected]>
Reference : Without this the output does not include Topics , Namespaces and Classes. Signed-off-by: Adarsh <[email protected]>
Signed-off-by: Adarsh <[email protected]>
Head branch was pushed to by a user without write access
7e9b942
to
f0ed56d
Compare
What
Reordered the sequence of topics in generated documentation. View mockup for Changes live here
Removed following outdated pages from documentation, until they are updated.
Why
Updated sequence of files listed by Doxygen output based on discussion in
Extra Reference
Current Sequence
Updated Sequence LIVE