Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Sep 30, 2025

Summary

Same as #55260 , for CI

prepare to add json output

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
… methods

- Extract collectDelegationData() method to eliminate code duplication
- Add formatSettingsData() method for consistent data formatting
- Add outputPlainFormat() method to separate concerns
- Add proper empty state handling with user-friendly message

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
- Add --output option supporting plain, json, and json_pretty formats
- Add validateOutputFormat() method for input validation
- Implement proper error handling for invalid output formats
- Support empty state handling for both plain and JSON outputs
- Use Base class writeArrayInOutputFormat() for consistent JSON output
- Maintain backward compatibility with plain format as default

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
@come-nc come-nc self-assigned this Sep 30, 2025
@come-nc come-nc requested a review from a team as a code owner September 30, 2025 09:26
@come-nc come-nc added the 2. developing Work in progress label Sep 30, 2025
@come-nc come-nc requested review from CarlSchwan, icewind1991 and yemkareems and removed request for a team September 30, 2025 09:26
@come-nc come-nc changed the title Feature/add json output to occ admin delegation local: admin-delegation occ - add output option for show command to support JSON formats Sep 30, 2025
@sorbaugh sorbaugh closed this pull request by merging all changes into master in f8fd111 Sep 30, 2025
@sorbaugh sorbaugh deleted the feature/add_json_output_to_occ_admin-delegation branch September 30, 2025 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants