nf-osi / NF_data_curator

Data Ingest Shiny App
https://sagebio.shinyapps.io/NF_data_curator/
Apache License 2.0
1 stars 1 forks source link

Add CI workflow with model repo, related to to issue #122 #133

Closed anngvu closed 2 years ago

anngvu commented 2 years ago

@allaway You can see the working chained workflows here:

This is currently a draft because I need to confirm/resolve two things for NF:

  1. For workflows to trigger other workflows, one needs to use a PAT instead of the default action token. See https://github.com/ad-m/github-push-action/issues/32. For GF, I set up a bot account, same idea as https://github.com/google-github-actions-bot.
  2. The PAT needs to have repo scope and be available to this repo -- so we can use REPO_PAT already used for shiny-deploy or set up something new. Once that's confirmed, the workflow can refer to the correct secret.
allaway commented 2 years ago

We have a bot account, @nfosi-service. I'll share the credentials with you on LP! I cannot recall whether a PAT already exists in LP but probably best to just create a new one anyway...

nfosi-service commented 2 years ago

Alright, I just set @.*** as the default email address for 2FA emails, and shared the creds with you on LP!

robert allaway (he/him) principal scientist sage bionetworks seattle, wa

On Tue, May 17, 2022 at 9:20 AM Robert Allaway @.***> wrote:

We have a bot account, @nfosi-service https://github.com/nfosi-service. I'll share the credentials with you on LP! I cannot recall whether a PAT already exists in LP but probably best to just create a new one anyway...

— Reply to this email directly, view it on GitHub https://github.com/nf-osi/NF_data_curator/pull/133#issuecomment-1129068127, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANWYPLFY663EDE5YBOWQ4SLVKPBLDANCNFSM5WBT4GCQ . You are receiving this because you were mentioned.Message ID: @.***>

allaway commented 2 years ago

Thanks, this is great! Looking forward to seeing this in action!