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 #272, Apply fix for Osal guide pdf generation #273

Conversation

nmullane
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution

Testing performed
Steps taken to test the contribution:

  1. Modified the if statements in build-documentation.yml to force the Osal pdf to generate on my forked branch to ensure the fix worked properly. The output of this GitHub Actions Documentation workflow run is here
  2. The change to the if statements was reverted and a GitHub Actions Documentation worfklow run without this modification is here

Expected behavior changes
A clear and concise description of how this contribution will change behavior and level of impact.

  • Changed a single cd command in build-documentation.yml which should let the subsequent make call run properly
  • No expected behavior should change. This only fixes an issue with the Osal guide not being generated by GitHub Actions.

System(s) tested on

  • GitHub actions ubuntu-18.04 runner

Code contributions
The cFS repository is provided to bundle the cFS Framework. It is utilized for bundling submodules, continuous integration testing, and version management and does not contain any software. Code contributions should be directed to the appropriate submodule.

Contributor Info - All information REQUIRED for consideration of pull request
Niall Mullane, GSFC Code 582 intern

@astrogeco astrogeco added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Jun 23, 2021
@astrogeco astrogeco marked this pull request as ready for review June 23, 2021 16:06
@astrogeco
Copy link
Contributor

astrogeco commented Jun 23, 2021

CCB:2021-06-23, APPROVED

  • double check that the cFE pdf is also being published

@astrogeco astrogeco changed the base branch from main to integration-candidate June 23, 2021 19:06
@astrogeco astrogeco force-pushed the integration-candidate branch 2 times, most recently from 0d2cd48 to 1afe0e9 Compare June 24, 2021 23:34
@astrogeco astrogeco merged commit b391b52 into nasa:integration-candidate Jun 24, 2021
@skliper
Copy link
Contributor

skliper commented Jun 28, 2021

@astrogeco I don't see updated pdfs https://github.com/nasa/cFS/tree/gh-pages. Still broken?

@astrogeco
Copy link
Contributor

astrogeco commented Jun 28, 2021

yep, "check for duplicates" is preventing this from running for the push job. Will need to modify the workflow

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub action workflow fails to generate Osal guide pdf
3 participants