- Notifications
You must be signed in to change notification settings - Fork673
-
I am trying to probe thehttps://docs.gitlab.com/ee/api/merge_requests.html#get-single-mr API, and I thought I'd use After some trying, I got to gitlab -v --gitlab inst-gitlab project-merge-request get --project-id 15 --iid 15899 I expected "some kind of output". However, I only received I needed to add ... However, why isn't that the default? I don't want to start a "default output format" religious war - butsome kind of output, that's not what I needed to spoon-feed in the first place What am I missing here? |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 1 comment 3 replies
-
@stdedos in the legacy output (the default), the default behavior is to print out the ID and what we call the I'm not sure what the decision behind the short vs. verbose behavior was as it's before my time. YAML would probably make more sense and just print it all out, but that pulls in another (currently optional) dependency for the CLI. |
BetaWas this translation helpful?Give feedback.
All reactions
-
... or JSON and avoid the extra dependency 🙃 Could we extend the config file with a global (and per-server?) option |
BetaWas this translation helpful?Give feedback.
All reactions
-
I guess JSON is more for programmatic use so might not be the nicest output by default 😅 Yes, a global config option might be best actually. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I just "don't want to remember this" anymore 😅 Maybe I'm a minority, but I'm using the CLI solely as the "quick" version of Legacy format (if extended) may make sense. Idk the amount of "ready formats" vs "not-ready" (like my command's case). If not-ready is a majority, it's better IMHO to switch to YAML (if you are fine for the dependency). Honestly, I was "mislead" thinking that I can pass posargs to the command and get the keys I wanted 😅 |
BetaWas this translation helpful?Give feedback.