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.

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

Fix #150, update doxygen comments to fix warnings #152

Merged

Conversation

lbleier-GSFC
Copy link
Contributor

@lbleier-GSFC lbleier-GSFC commented Apr 10, 2020

Describe the contribution
Fixes #150
Resolve doxygen warnings

Testing performed
Steps taken to test the contribution:

  1. Corrected lines that generated warnings
  2. Rebuilt documentation with make doc
  3. Observed no warnings generated
  4. Viewed relevant page(s) to verify correctness

Expected behavior changes
Changes to documentation only; no code impact

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

@skliper skliper added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Apr 10, 2020
@skliper
Copy link
Contributor

skliper commented Apr 10, 2020

Not sure why these aren't autolinking to the issue... try:
Fix #150
Resolve doxygen warnings

or any other keyword as described in https://help.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue

@skliper skliper linked an issue Apr 15, 2020 that may be closed by this pull request
@skliper
Copy link
Contributor

skliper commented Apr 15, 2020

CCB 20200415 - APPROVED

@astrogeco astrogeco added CCB:Approved Indicates Approval by CCB CCB - 20200415 docs and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Apr 20, 2020
@astrogeco astrogeco merged commit ab38fe6 into nasa:integration-candidate Apr 21, 2020
@skliper skliper added this to the 1.5.0 milestone Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates Approval by CCB docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix doxygen warnings
4 participants