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

Update ingest/vendored #130

Merged
merged 1 commit into from
Mar 3, 2025
Merged

Update ingest/vendored #130

merged 1 commit into from
Mar 3, 2025

Conversation

joverlee521
Copy link
Contributor

Description of proposed changes

Pull in latest vendored nextstrain/ingest scripts to update notify-slack.

Related issue(s)

Part of nextstrain/ingest#46
Resolves #103

Checklist

  • Checks pass

@joverlee521
Copy link
Contributor Author

Note that the other changes in the vendored scripts do not affect the workflows in this repo since we only use 4 of the vendored scripts:

@joverlee521
Copy link
Contributor Author

Ah, the update in this repo is not really necessary since we don't use the --upload flag with notify-slack.
Oh well, doesn't hurt to update and prevent Slack errors from causing workflows to fail.

subrepo:
  subdir:   "ingest/vendored"
  merged:   "cd6d31a"
upstream:
  origin:   "https://github.com/nextstrain/ingest"
  branch:   "main"
  commit:   "cd6d31a"
git-subrepo:
  version:  "0.4.6"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "110b9eb"
@joverlee521
Copy link
Contributor Author

Will merge on Monday before the automated workflows run.

@joverlee521 joverlee521 merged commit 769d23e into main Mar 3, 2025
1 check passed
@joverlee521 joverlee521 deleted the update-vendored branch March 3, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ingest: Update vendored scripts
1 participant