You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I reviewed the cFS README.md file to see if the feature is in the major future work.
I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.
Is your feature request related to a problem? Please describe.
The deploy jobs in the documentation workflow run even when the criteria for deployment are not met. This is not a problem since the git push step, which does the actual deployment does not run.
However, this does spin up an extra VM that then gets shut down immediately. Skipping the job can shave a couple of seconds from the full CI suite.
Moves the branch push check conditional "up" in the documentation build
workflow. The check now happens at the deploy job level instead of the
deploy step step for both the deploy-usersguide and deploy-osalguide.
This saves time by preventing loading extra VMs in pull requests and
other repository push events.
Checklist (Please check before submitting)
Is your feature request related to a problem? Please describe.
The deploy jobs in the documentation workflow run even when the criteria for deployment are not met. This is not a problem since the git push step, which does the actual deployment does not run.
However, this does spin up an extra VM that then gets shut down immediately. Skipping the job can shave a couple of seconds from the full CI suite.
cFS/.github/workflows/build-documentation.yml
Line 271 in 4ad0677
Describe the solution you'd like
Move the if condition up one level from the deploy step to the deploy job.
Describe alternatives you've considered
Leave as is.
Additional context
None
Requester Info
Full name and company/organization if applicable
The text was updated successfully, but these errors were encountered: