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

Docs not deployed when PR is merged to main #463

Closed
2 tasks done
astrogeco opened this issue Apr 19, 2022 · 0 comments · Fixed by #466
Closed
2 tasks done

Docs not deployed when PR is merged to main #463

astrogeco opened this issue Apr 19, 2022 · 0 comments · Fixed by #466

Comments

@astrogeco
Copy link
Contributor

astrogeco commented Apr 19, 2022

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug

The skip duplicates check catches the "same content" from pull requests and thus isn't being triggered by a push to main.

To Reproduce

See workflow runs related to IC: Caelum-rc4+dev8.

https://github.com/nasa/cFS/actions/runs/2192037376, push to main, skipped due to run in PR #461 https://github.com/nasa/cFS/actions/runs/2191974505

Expected behavior

Always run the documentation job on a push to main.

Proposed Solution

Skip the "check-for-duplicates" step in pushes to main branch.

System observed on:
Github actions

Additional context
n/a

Reporter Info
Gerardo E. Cruz-Ortiz, NASA

@astrogeco astrogeco changed the title Documentation doesn't build when PR is merged to main DOxygen docs not deployed when PR is merged to main Apr 19, 2022
@astrogeco astrogeco changed the title DOxygen docs not deployed when PR is merged to main Docs not deployed when PR is merged to main Apr 19, 2022
@astrogeco astrogeco added the good first issue Good for newcomers label Apr 19, 2022
skliper added a commit to skliper/cFS that referenced this issue Apr 21, 2022
skliper added a commit to skliper/cFS that referenced this issue Apr 21, 2022
astrogeco added a commit that referenced this issue Apr 25, 2022
havencarlson pushed a commit to havencarlson/cFS that referenced this issue Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant