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.

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

Inconsistent date/time handling in CLIs #1707

Open
tjtg opened this issue May 2, 2022 · 0 comments
Open

Inconsistent date/time handling in CLIs #1707

tjtg opened this issue May 2, 2022 · 0 comments

Comments

@tjtg
Copy link
Contributor

tjtg commented May 2, 2022

As encountered in #1705 (comment), there are a variety of ways of handling CLIs with date/time inputs.

  • Most are timezone naive, a few enforce use of the UTC "Z" timezone
  • Some CLIs pass these onto internal processing functions as strings, others parse the datetime early and pass a python datetime.datetime object
  • Some CLIs use lists of datetimes and this usage will need to be catered for

It would be desirable for the CLIs to use a single consistent approach to dealing with date/time inputs, rather than the current variety of approaches. See summary written by @benowen-bom in #1705 (comment) for further details of CLIs with date/time inputs.

Acceptance criteria:

  • Agreed approach for handling date/time inputs to CLIs
  • Code changes so that CLIs which are passed date/time strings as input use the agreed approach
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant