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.

Sign up for GitHub

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

Jump to bottom

export: Allow --node-data to be repeated to build up a file list #1010

Merged
merged 3 commits into from
Sep 19, 2022
Merged

export: Allow --node-data to be repeated to build up a file list #1010

merged 3 commits into from
Sep 19, 2022
+38 −2

Commits on Sep 19, 2022

  1. tests: Failing test showing repeated --node-data options to augur exp… 

    …ort v2 overwrite each other
    @tsibley
    tsibley committed Sep 19, 2022
    Configuration menu
    Copy the full SHA
    d0cb26e View commit details
    Browse the repository at this point in the history

  2. export: Allow --node-data to be repeated to build up a file list 

    …instead of each repetition overriding the last values.  While there is
some theoretical utility in being able to override one list of node data
files with another in a subsequent option, I think it's more likely that
repetition is intended to built up a single list.  I was surprised, at
least.

See also <https://discussion.nextstrain.org/t/1194/4>.
    @tsibley
    tsibley committed Sep 19, 2022
    Configuration menu
    Copy the full SHA
    e5eefa4 View commit details
    Browse the repository at this point in the history

  3. Update changelog

    @tsibley
    tsibley committed Sep 19, 2022
    Configuration menu
    Copy the full SHA
    de8147b View commit details
    Browse the repository at this point in the history