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

Replacing artifact and setting retention days to 7 #230

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 17 additions & 23 deletions .github/workflows/pkgdown.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,8 @@ jobs:
uses: actions/upload-pages-artifact@v3
with:
path: ./docs/
name: github-pages
retention-days: 7

deploy:
# check builds on PRs but only deploy when main changes
Expand All @@ -190,31 +192,23 @@ jobs:
uses: actions/deploy-pages@v4

post-page-artifact:
# only comment on PRs
runs-on: ubuntu-latest

if: ${{ github.event_name == 'pull_request' }}

# This job depends on the `build` job
needs: combine
runs-on: ubuntu-latest

# Required permissions
permissions:
contents: read
pull-requests: write
env:
GH_TOKEN: ${{ github.token }}
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Find Comment
uses: peter-evans/find-comment@v3
id: fc
with:
issue-number: ${{ github.event.pull_request.number }}
comment-author: 'github-actions[bot]'
body-includes: Your page is ready to preview
contents: read
pull-requests: write

- name: Create or update comment
uses: peter-evans/create-or-update-comment@v4
steps:
# Post the artifact pulling the id from the `readme` step.
# The msg will refer to the arfitact as 'README file'.
- name: Post the artifact
uses: CDCgov/cfa-actions/[email protected]
with:
comment-id: ${{ steps.fc.outputs.comment-id }}
issue-number: ${{ github.event.pull_request.number }}
body: |
Thank you for your contribution, @${{ github.triggering_actor }} :rocket:! Your page is ready to preview [here](https://github.com/${{github.repository}}/actions/runs/${{ github.run_id }}/artifacts/${{ needs.build.outputs.page_artifact_id }})
edit-mode: replace
artifact-name: github-pages
gh-token: ${{ secrets.GITHUB_TOKEN }}
5 changes: 4 additions & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# wwinference dev
# wwinference 0.1.0.99 (dev)

## Internal changes

- Updated the workflow for posting the pages artifact to PRs (issue [#229](https://github.com/CDCgov/ww-inference-model/issues/229)).
- Modify `plot_forecasted_counts()` so that it does not require an evaluation dataset ([#218](https://github.com/CDCgov/ww-inference-model/pull/218))

# wwinference 0.1.0
Expand Down
Loading