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 #312, Generate Documentation In All Scenarios #316

Conversation

ArielSAdamsNASA
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
Fixes #312

remove if: ${{ github.event_name == 'push' && contains(github.ref, 'main')}} from PDF generation installs and PDF generation. Add PDF generation installs and PDF generation steps to the previous job, so that Deploy is a separate job.

Testing performed
N/A

Expected behavior changes
PDFs will generate no matter what while deployment is based on if it is pushed on main.

Additional context
May require cache PR for successful deployment: #315

Contributor Info - All information REQUIRED for consideration of pull request
Ariel Adams, ASRC Federal

arielswalker and others added 2 commits July 22, 2021 14:29
…d-Documentation-Workflow

Fix nasa#310, No File Found Documentation Workflow

Renamed jobs and steps for clarity
Minor edits to step order in deploy jobs
@ArielSAdamsNASA ArielSAdamsNASA added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Jul 23, 2021
@ArielSAdamsNASA ArielSAdamsNASA marked this pull request as draft July 26, 2021 18:28
@ArielSAdamsNASA ArielSAdamsNASA force-pushed the fix-312-generate-documentation-in-all-scenarios branch 2 times, most recently from 2433f08 to a00547b Compare July 26, 2021 19:24
@ArielSAdamsNASA ArielSAdamsNASA marked this pull request as ready for review July 26, 2021 19:24
@ArielSAdamsNASA ArielSAdamsNASA force-pushed the fix-312-generate-documentation-in-all-scenarios branch from a00547b to 99a13ba Compare July 26, 2021 19:25
@ArielSAdamsNASA ArielSAdamsNASA force-pushed the fix-312-generate-documentation-in-all-scenarios branch from 99a13ba to 258f1df Compare July 27, 2021 19:07
@ArielSAdamsNASA ArielSAdamsNASA force-pushed the fix-312-generate-documentation-in-all-scenarios branch from 258f1df to 1cb05f2 Compare July 27, 2021 19:09
@astrogeco astrogeco changed the base branch from main to integration-candidate July 29, 2021 19:08
@astrogeco astrogeco added 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 Jul 29, 2021
@astrogeco astrogeco merged commit ccc754f into nasa:integration-candidate Jul 29, 2021
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 docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generate documentation PDF in all scenarios regardless of whether it will be pushed or not.
3 participants