-
Notifications
You must be signed in to change notification settings - Fork 140
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[mqba]: Complete conversion of mqba documentation from BDE style to Doxygen style #523
Conversation
93af0db
to
51a7e26
Compare
51a7e26
to
88c4362
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.
Build 394 of commit 88c4362 has completed with FAILURE
88c4362
to
0632f3d
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.
Build 423 of commit 0632f3d has completed with FAILURE
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.
Looks solid 👍
This patch removes the old `doc/*.txt` file and migrates some information in it into a file Doxygen can read. The information not migrated is: 1. the table of contents for this package, which will be generated by Doxygen automatically, and kept in-sync with any documentation changes made in the files, and 2. the hierarchical synopsis, which is out-of-date. These are the same changes that we have already made in the public documentation, as in commits bc20ab0 and f3ca1d2. See in particular the rational in the latter’s commit message. Signed-off-by: Patrick M. Niedzielski <[email protected]>
This patch makes three changes: 1. First, it converts the BDE-style component documentation into Doxygen `@file` documentation. See the commit message in commit f3ca1d2 for more details on this. 2. Next, it converts stray BDE-style documentation that was not automatically converted to Doxygen format when we open-sourced. Most of these are the documentation for member variables, but in some cases, overloads were not correctly converted either. 3. Finally, it makes minor changes to non-Doxygen comments, like file header comments or include comments, which were not correctly updated in the past. Signed-off-by: Patrick M. Niedzielski <[email protected]>
0632f3d
to
1163f5e
Compare
🎉 |
Please see commit messages for details.
This is the first of many PRs like this. They will all follow the same format (two commits, one to convert component documentation and to fix up things that the Doxygen conversion script missed, and one to migrate package documentation.