-
Notifications
You must be signed in to change notification settings - Fork 129
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
Remove export v1
#1266
Comments
Noting that we still use |
@joverlee521 that's good to know. Do you expect that those builds would benefit from newer features of Augur? Or can it also vendor its own version of Augur, similar to its current usage of Auspice? It doesn't seem sensible to keep |
Hmm, maybe we could vendor its own version of |
Let's at least add the deprecation warning. It's good practice to leave things around for a while after the warning appears, so that'll give us plenty of time to decide what to do re: seasonal flu. In hindsight, having a sub-(sub-?)command wasn't one of my best ideas so perhaps we could explore things like making |
I started on this a while back with db54927...ec16b56 but it's probably stale now. James has started a new PR. |
Prompted by #1262 (comment).
Description
export v2
(for Auspice v2) has been around for a while, and should be widely adopted at this point. There should no longer be a need to maintainexport v1
in newer versions of Augur.Tasks
export v1
. export v1: Add deprecation warning, remove--metadata-id-columns
#1265export v1
.export v2
toexport
with a command line "redirect" in place. (maybe not necessary)The text was updated successfully, but these errors were encountered: