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 #234, Documentation and Guides Workflow Run on All Push Events #236

Merged
merged 1 commit into from
Apr 22, 2021
Merged

Fix #234, Documentation and Guides Workflow Run on All Push Events #236

merged 1 commit into from
Apr 22, 2021

Conversation

ArielSAdamsNASA
Copy link
Contributor

@ArielSAdamsNASA ArielSAdamsNASA commented Apr 6, 2021

Checklist (Please check before submitting)

Describe the contribution
Fixes #234

  • Removed the main branch for push so that the workflow runs on all branches for push and pull requests.
  • Added the duplicate jobs action so that the workflow does not run any duplicate jobs.

Testing performed
Tested locally

Expected behavior changes

  • The Documentation and Guides workflow should run on all push events. For example, when users push code to their forked repos, the Documentation and Guides workflow should run.
  • When a user pushes code to a pull request, the workflow should not run duplicate jobs for both push and pull requests.

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

@ArielSAdamsNASA ArielSAdamsNASA added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) continuous-integration labels Apr 6, 2021
@astrogeco
Copy link
Contributor

CCB:2021-04-07 APPROVED

@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 Apr 7, 2021
@skliper skliper added CCB:2021-04-07 and removed CCB:Approved Indicates code review and approval by community CCB labels Apr 14, 2021
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.

Documentation and Guides workflow doesn't run for all push events
4 participants