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
Mention Code Complete as a resource. #10898
Conversation
I'm reading this book for a second time in its entirety with my research group, and I continue to like it a lot. It's worth mentioning. |
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 good to me but I have an optional improvement.
So done! |
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.
I double checked the doxygen output and we need to fix some minor things.
examples/step-1/doc/intro.dox
Outdated
resource to this end is the book "<a | ||
href="https://en.wikipedia.org/wiki/Code_Complete">Code | ||
Complete</a>" by Steve McConnell @code CodeComplete . It's already |
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.
resource to this end is the book "<a | |
href="https://en.wikipedia.org/wiki/Code_Complete">Code | |
Complete</a>" by Steve McConnell @code CodeComplete . It's already | |
resource to this end is the book | |
<a href="https://en.wikipedia.org/wiki/Code_Complete">"Code Complete"</a> | |
by Steve McConnell @cite CodeComplete. It's already |
doxygen just prints the link literally instead of including the HTML when the whole link is quoted for me. We definitely need @cite
and not code
.
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.
Let me do this via markdown. Does this work for you now?
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.
Almost - you forgot to delete @code
and add @cite
.
Fixed now! |
No description provided.