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

Ability to use Command line tool PD to do all #463

Open
zerolaser opened this issue Sep 10, 2022 · 2 comments
Open

Ability to use Command line tool PD to do all #463

zerolaser opened this issue Sep 10, 2022 · 2 comments
Labels

Comments

@zerolaser
Copy link

  1. Currently the pd service list only list 25 entries. Ability to loop all the entries via command line.
  2. Ability to print output as txt, json or csv.
  3. What's the debug option when running via commandline ?
@jsleeio
Copy link
Contributor

jsleeio commented Feb 8, 2024

I'm working on a PR for optional JSON support. I have it working for all commands but before I submit I'd like to understand why some of them use JSON and the rest use YAML.

My change introduces a Meta.Marshaler struct field which has a value of either yaml.Marshal or json.Marshal, and configures this via a global option. It seems to work nicely.

However ... if this change is merged, regardless of what the default output format is, it will break scripted pd usage. Even not having a default at all will break scripted pd usage because people will now need to configure it.

Maintainer people? What do you think?

@jsleeio
Copy link
Contributor

jsleeio commented Feb 8, 2024

PR: #513

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants