Skip to content

docs(lib-c): Add doxygen comments for lib-c #1184

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Oct 17, 2018
Merged

docs(lib-c): Add doxygen comments for lib-c #1184

merged 3 commits into from
Oct 17, 2018

Conversation

GMishx
Copy link
Member

@GMishx GMishx commented Aug 28, 2018

Description

Added missing doxygen comments for FOSSology's C library.

How to test

Current documentation: https://gmishx.github.io/ (updated on every commit).

Please merge after #1174

@ghost ghost assigned GMishx Aug 28, 2018
@ghost ghost added the needs code review label Aug 28, 2018
@GMishx GMishx force-pushed the docs/doxygen-libc branch 2 times, most recently from 63001d2 to 3991e7f Compare August 29, 2018 06:37
@mcjaeger mcjaeger added the has merge conflicts PR to be rebased label Sep 14, 2018
@GMishx GMishx removed the has merge conflicts PR to be rebased label Sep 14, 2018
@GMishx GMishx force-pushed the docs/doxygen-libc branch 5 times, most recently from ae0f45d to 28180b7 Compare September 18, 2018 07:37
@mcjaeger mcjaeger self-assigned this Sep 21, 2018
@mcjaeger mcjaeger self-requested a review October 17, 2018 19:15
mcjaeger
mcjaeger approved these changes Oct 17, 2018
View reviewed changes
Copy link
Member

@mcjaeger mcjaeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nothing serious

@mcjaeger mcjaeger merged commit d66b1cf into fossology:master Oct 17, 2018
@ghost ghost removed the needs code review label Oct 17, 2018
@GMishx GMishx deleted the docs/doxygen-libc branch October 18, 2018 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcjaeger mcjaeger mcjaeger approved these changes

Assignees

@mcjaeger mcjaeger

@GMishx GMishx

Labels

depends on other PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@GMishx @mcjaeger