Skip to content
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

Fix #447, Resolve doxygen warnings #451

Merged
merged 1 commit into from
May 26, 2020
Merged

Fix #447, Resolve doxygen warnings #451

merged 1 commit into from
May 26, 2020

Conversation

lbleier-GSFC
Copy link
Contributor

Describe the contribution
Fixes #447, additional doxygen warnings

Testing performed
Steps taken to test the contribution:

  1. Corrected the issues referenced in the warnings.log file
  2. Rebuilt the docs with make doc
  3. Observed no recurrence of same warnings in warnings.log

Expected behavior changes
No doxygen warnings for the affected files

Contributor Info - All information REQUIRED for consideration of pull request
Leor Bleier, NASA GSFC\Code 582

@lbleier-GSFC lbleier-GSFC changed the base branch from master to integration-candidate May 8, 2020 13:32
@skliper skliper changed the title Fix #447 - more doxygen warnings Fix #447, Resolve doxygen warnings May 11, 2020
@skliper skliper added this to the 5.1.0 milestone May 16, 2020
@astrogeco astrogeco added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label May 19, 2020
@astrogeco
Copy link
Contributor

astrogeco commented May 20, 2020 •
edited
Loading

CCB 20200520 - Approved

@skliper skliper added CCB-20200520 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels May 20, 2020
@astrogeco astrogeco added CCB:Approved Indicates code review and approval by community CCB IC-20200520 labels May 26, 2020
@astrogeco astrogeco merged commit a66770f into nasa:integration-candidate May 26, 2020
@astrogeco astrogeco mentioned this pull request May 29, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CDKnightNASA CDKnightNASA CDKnightNASA approved these changes

Assignees
No one assigned
Labels
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Milestone
Bootes
Development

Successfully merging this pull request may close these issues.

More doxygen warnings
4 participants
@lbleier-GSFC @astrogeco @CDKnightNASA @skliper