Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-27462

Add --config-file option alias to CmdLineTask

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP F20-6 (November), AP S21-3 (February)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      CmdLineTask uses --configfile to specify task config files, while pipetask uses --config-file. This is a potentially confusing change in UIs. Allow CmdLineTask users to specify --config-file in addition to the old option name, so that users can switch to the new convention quickly.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-22663 Reimplement make_apdb.py for Gen 3

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment - - edited

            Any preferences on how deep the changes should go?

            • Add a --config-file alias and call it a day.
            • Also update references to --configfile in other --help text to say --config-file.
            • Also update references in the Sphinx docs.

            Just adding the alias seems kind of pointless, but the broader changes are more likely to confuse existing users...

            Show
            krzys Krzysztof Findeisen added a comment - - edited Any preferences on how deep the changes should go? Add a --config-file alias and call it a day. Also update references to --configfile in other --help text to say --config-file . Also update references in the Sphinx docs. Just adding the alias seems kind of pointless, but the broader changes are more likely to confuse existing users...
            Hide
            Permalink
            mrawls Meredith Rawls added a comment -

            If it's quick to do all of these, that would be nicer, but if it's not, I'd accept just the alias TBH since we shouldn't spend too much time on Gen2 work.

            But if it's a quick and easy update, all three of your bullets would be nice... every instance of -configfile should be pretty easily swapped for "-config-file or --configfile"

            Is it still -C (capitol) when specifying a file and -c (lowercase) when not? That's another potential gotcha.

            Show
            mrawls Meredith Rawls added a comment - If it's quick to do all of these, that would be nicer, but if it's not, I'd accept just the alias TBH since we shouldn't spend too much time on Gen2 work. But if it's a quick and easy update, all three of your bullets would be nice... every instance of - configfile  should be pretty easily swapped for " -config-file or --configfile " Is it still -C (capitol) when specifying a file and -c (lowercase) when not? That's another potential gotcha.
            Hide
            Permalink
            price Paul Price added a comment -

            Great!

            Show
            price Paul Price added a comment - Great!

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Paul Price
              Watchers:
              Krzysztof Findeisen, Meredith Rawls, Paul Price
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.