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

Integration Candidate 20200121 #486

Closed
wants to merge 57 commits into from
Closed

Integration Candidate 20200121 #486

wants to merge 57 commits into from

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 21, 2020

Describe the contribution
Fix #459, fix #355, fix, #24, fix #101, fix #412, fix #189

Testing performed
Steps taken to test the contribution:

  1. Current CI passes
  2. Enhanced CI in Add "const" to function prototypes where appropriate #40 passes debug/with deprecated elements testing

Expected behavior changes

  • Resolve doxygen warnings
  • add OMIT_DEPRECATED flag
  • Add strict warnings and enforce
  • Always increment sequence counter
  • Fix comments
  • Add code coverage test to evs

System(s) tested on

  • CI: Ubuntu 18.04

Additional context
N/A

Third party code
N/A

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper
Copy link
Contributor Author

skliper commented Jan 23, 2020

@dmknutsen yikes, don't do that!

@dmknutsen
Copy link
Contributor

Apologies. I was experimenting with my fork...and accidentally force-pushed here instead. Wont happen again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment