nextstrain / ncov-ingest

A pipeline that ingests SARS-CoV-2 (i.e. nCoV) data from GISAID and Genbank, transforms it, stores it on S3, and triggers Nextstrain nCoV rebuilds.
MIT License
36 stars 20 forks source link

Remove unused notify-on-metadata-change #417

Closed victorlin closed 1 year ago

victorlin commented 1 year ago

Description of proposed changes

This has been unused for over 2 years now, and other pathogen repos are using a new script notify-on-diff¹ that does not have memory issues.

Instead of replacing notify-on-metadata-change with notify-on-diff, remove it entirely given that thing are working fine with the other output files being diffed for important changes.

¹ https://github.com/nextstrain/ingest/blob/1eb8b30428d5f66adac201f0a246a7ab4bdc9792/notify-on-diff

Related issue(s)

Prompted by https://github.com/nextstrain/ingest/pull/12#discussion_r1284902660.

Testing