-
Notifications
You must be signed in to change notification settings - Fork 0
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
[CI] Set up deploy
workflow for GitHub pages
#52
base: main
Are you sure you want to change the base?
Conversation
Reviewer's Guide by SourceryThis pull request introduces a new GitHub Actions workflow ( No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
✅ Deploy Preview for staging-annotation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #52 +/- ##
=======================================
Coverage 80.48% 80.48%
=======================================
Files 17 17
Lines 123 123
Branches 17 17
=======================================
Hits 99 99
Misses 24 24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
🧑🍳
# TODO: Update/uncomment once we have deploy the app to replace the legacy tool | ||
# - name: Add prod scripts | ||
# run: | | ||
# add_script='<script async src="https://w.appzi.io/w.js?token=YVTHd"></script><script defer data-domain="annotate.neurobagel.org" src="https://plausible.neurobagel.org/js/script.js"></script>' | ||
# # Need to use single quotes around sed to avoid ! to be interpreted as a history expansion character | ||
# sed -i 's|<!-- REPLACE_ME_NB_ANNOTATE_HEADER_SCRIPT_REPLACE_ME -->|'"${add_script}"'|' index.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
Checklist
This section is for the PR reviewer
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
,[MODEL]
,[DOC]
) (see our Contributing Guidelines for more info)skip-release
(to be applied by maintainers only)Closes #XXXX
For new features:
For bug fixes:
Summary by Sourcery
Sets up a
deploy
workflow for GitHub pages. This workflow automates the deployment process to GitHub Pages upon the creation of a new release.CI:
Deployment: