Fix Uint32Slice param array to accept empty strings #1495
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Accept empty strings in flag/env parsing by adding
config.Uint32Sliceand using it forconfig.PayerOptions.NodeSelectorPreferredNodesAdd
config.Uint32SlicewithUnmarshalFlagandStringfor comma-separateduint32parsing, and switchPayerOptions.NodeSelectorPreferredNodesto this type in options.go. Parsing trims spaces, skips empty tokens, treats empty input as nil, and errors on non-uint32tokens;Stringreturns comma-separated values.📍Where to Start
Start with
config.Uint32Slice.UnmarshalFlagin custom_types.go, then review the field change in options.go.Macroscope summarized b39f6b0.