fullstorydev / fullstorydev/grpcurl
Change -emit-defaults behaviour so that the flag isn't anymore needed to show default values (Proto3)
- Dominant language
- Go
- Stars
- 12.8k
- Forks
- 580
- Avg merge
- 1h 8m
- Merged PRs (30d)
- 5
Description
Hi,
I encountered the same issue as described in #97.
After some debugging, I tried to call the ame service with BloomRPC GUI and default value was shown immediately.
Some research lead me to -emit-defaults option which solves the issue for me.
Although this is expected behaviour as explained in #97, it would be desirable to have -emit-defaults set by default and give the user the option to omit defaults if needed.
This is obviously how BloomRPC and other clients/client libraries work.
Or is there anything against this approach?
Thanks!
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the grpcurl command-line handling for the -emit-defaults flag and tracing how default values are emitted. Define the default behavior and the option for omitting defaults, then verify the CLI behavior against the Proto3 case described in the issue; no specific file or test is named.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100