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

cmd/status: endpoint implementation #42

Merged
merged 6 commits into from
Oct 6, 2020
Merged

cmd/status: endpoint implementation #42

merged 6 commits into from
Oct 6, 2020

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Oct 5, 2020

Initial implementation of the /status endpoint. With colors on the table display

output

Closes: #27

@tormath1 tormath1 self-assigned this Oct 5, 2020
@gaelL
Copy link
Member

gaelL commented Oct 5, 2020

My2cents Could be great to convert it into something we could use in scripts with json output.
Or adding some colors in the tab view

@xlr-8
Copy link
Contributor

xlr-8 commented Oct 5, 2020

Generally speaking, I think it would be nice to have formatting (which in the end do not format anything received 😄 ) flags. --json|yaml as extras

@tormath1
Copy link
Contributor Author

tormath1 commented Oct 5, 2020

Generally speaking, I think it would be nice to have formatting (which in the end do not format anything received smile ) flags. --json|yaml as extras

What do you mean ? We already have the --output/-o yaml|json|table 😁

$ ./cy status -o json | jq
Warning: CLI version v1.0.48 does not match the API version. You should consider to download CLI version v1.0.47

[
  {
    "canonical": "database",
    "category": "internal",
    "message": "no issue reported while connecting to the database service",
    "status": "Success"
  },
  {
    "canonical": "pipeline",
    "category": "internal",
    "message": "no issue reported while connecting to the pipeline service",
    "status": "Success"
  },
...
$ ./cy status -o yaml
Warning: CLI version v1.0.48 does not match the API version. You should consider to download CLI version v1.0.47

- canonical: database
  category: internal
  message: no issue reported while connecting to the database service
  status: Success
...

@xlr-8
Copy link
Contributor

xlr-8 commented Oct 7, 2020

What do you mean ? We already have the --output/-o yaml|json|table grin

Then I don't get @gaelL comment ^^

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 this pull request may close these issues.

cli/cmd: implement status command
3 participants