nextstrain / zika

Nextstrain build for Zika virus
https://nextstrain.org/zika
8 stars 10 forks source link

Use `ingest/scripts` to match `phylogenetic/scripts` #66

Closed j23414 closed 8 hours ago

j23414 commented 6 days ago

Description of proposed changes

While chatting about a standard directory structure of the pathogen repository, we noticed that zika was using both phylogenetic/scripts and ingest/bin.

This PR picks bin to keep things consistent.

Additionally, deleted an unused script.

Related issue(s)

Checklist

victorlin commented 6 days ago

This PR picks bin to keep things consistent.

I'm all for consistency! Noting that there is a split across @nextstrain:

If zika was just a one-off then I would suggest being consistent with other repos. But there's a clear lack of consistency across many repos. Let's open an issue in nextstrain/private where we can discuss and decide which to use, then update this PR + other repos with the decision?

j23414 commented 8 hours ago

Updated this PR with the decision to use "scripts" instead of "bin". Ready for review