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 #344, add doc-prebuild dependency #347

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

jphickey
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
The documentation requires some artifacts to be in place, this added dependency ensures they are created before doxygen runs.

Fixes #344

Testing performed
Run documentation build

Expected behavior changes
No missing file if this is the first/only target built (file will be created due to dependency)

System(s) tested on
Ubuntu 22.04

Additional context
Requires merging nasa/cFE#2214 and nasa/osal#1341 before this will actually work.

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

The documentation requires some artifacts to be in place, this added
dependency ensures they are created before doxygen runs.
@jphickey jphickey added CCB:Ready Ready for discussion at the Configuration Control Board (CCB) dependency labels Nov 25, 2022
@dzbaker dzbaker added CCB:Approved and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Dec 1, 2022
@dzbaker dzbaker merged commit b9a8898 into nasa:main Dec 1, 2022
@dmknutsen dmknutsen added this to the Draco milestone Jan 18, 2023
@jphickey jphickey deleted the fix-344-doc-dependency branch April 17, 2023 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing dependency in documentation build
3 participants