-
Notifications
You must be signed in to change notification settings - Fork 2
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
Sync descriptions and questions to delphi-epidata #1216
Sync descriptions and questions to delphi-epidata #1216
Conversation
✅ Preview link ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
very nice! have you tried this when the files already exist in the target repo? i presume cp
does not complain about overwriting them. also, if there is no change to the files, does the "create PR" step exit with a graceful no-op?
Co-authored-by: melange396 <george.haff@gmail.com>
Co-authored-by: melange396 <george.haff@gmail.com>
@melange396 I've applied the changes!
indeed! it just overwrites silently
Yep, that action doesn't create a pull request with |
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.
LGTM!
Prerequisites:
dev
branchdev
Summary
As part of the "Create Release" action in the
www-covidcast
repo, creates a PR in thedelphi-epidata
repo that updates (copies) two text files fromwww-covidcast
.Tested out in my fork of this repo. A link to a successful run can be found here, and the PR it created it is cmu-delphi/delphi-epidata#1154.
PREREQUISITE: Add a secret to this repo called
CMU_DELPHI_DEPLOY_MACHINE_PAT
, which is a personal access token with access to both this repo anddelphi-epidata
.