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 #2064, Use osal-common.doxygen to resolve OSAL Doxygen refs #2065

Merged
merged 2 commits into from
Mar 16, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Mar 16, 2022

Describe the contribution

Testing performed
Built osal users guide and cfe users guide and mission doc from bundle level, confirmed contents.

Expected behavior changes
Docs only, reduces dependencies

System(s) tested on

Additional context
nasa/osal#1332

Third party code
None

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

@skliper skliper added CCB:Ready Ready for discussion at the Configuration Control Board (CCB) dependency labels Mar 16, 2022
@skliper skliper added this to the Draco milestone Mar 16, 2022
@skliper skliper force-pushed the fix2064-doxygen_common_use branch from ffbe723 to 198ae7d Compare March 16, 2022 13:33
@astrogeco
Copy link
Contributor

astrogeco commented Mar 16, 2022

CCB:2022-03-16 APPROVED

  • Check that warnings directory in workflow is referenced correctly

@astrogeco astrogeco added CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Mar 16, 2022
@skliper
Copy link
Contributor Author

skliper commented Mar 16, 2022

0a9e232 - warning check fixed, should be ready for IC

astrogeco added a commit to nasa/cFS that referenced this pull request Mar 16, 2022
nasa/osal#1232, Expose osal common config
nasa/cFE#2065, Resolve OSAL Doxygen refs with osal-common config
@astrogeco astrogeco merged commit 848912c into nasa:main Mar 16, 2022
@skliper skliper deleted the fix2064-doxygen_common_use branch April 19, 2022 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code review and approval by community CCB dependency draco-rc1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use osal common config to resolve OSAL doxygen references and provide default settings
2 participants