Skip to content

Wrong Date format in version output #283

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

Closed
mastrolinux opened this issue Jul 16, 2019 · 0 comments · Fixed by #287
Closed

Wrong Date format in version output #283

mastrolinux opened this issue Jul 16, 2019 · 0 comments · Fixed by #287

Comments

@mastrolinux
Copy link
Contributor

mastrolinux commented Jul 16, 2019

Bug Report

The date output is not a RFC3339 format

Current behavior

Build date does not have an easy parseable date format like RFC3339
now the CLI output is:

./arduino-cli version --format json | jq .build_date
"2019-07-16 15:06:29.7368222 +0000 UTC"

Expected behavior

it was supposed to use: https://golang.org/pkg/time/#Time.Format
it should print something RFC3339 compatible, not a custom Go language format

Environment

  • CLI version (output of arduino-cli version):
    {
    "command": "arduino-cli",
    "version": "0.3.7-alpha.preview",
    "commit": "",
    "build_date": "2019-07-16 15:09:59.2173451 +0000 UTC"
    }
  • Go version (if building from sources):
    1.12.5
  • OS version:
    Windows and Linux

Additional context

This was needed to improve command line testing.

@masci masci added bug labels Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
u5surf added a commit to u5surf/arduino-cli that referenced this issue Jul 17, 2019
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

Successfully merging a pull request may close this issue.

2 participants