-
Notifications
You must be signed in to change notification settings - Fork 0
feat: ✨ implement azdo security group show command
#104
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…oduces FindGroupsByDisplayName on the extension client to return hydrated graph groups with case-insensitive matching. This unlocks the security group commands to resolve descriptors without reimplementing query logic.
…ndGroupsByDisplayName stub so tests can configure the extensions client when resolving security groups. Without this extension, the updated commands would panic when the mock is injected and the helper is called.
…tion\n\nCreates reusable parsing and descriptor utilities so all security group commands share consistent validation. This common layer eliminates duplicated parsing code and ensures descriptors resolve through the extensions client.
…tches scope parsing to the shared helpers and runs graph mutations with the command context. This keeps create aligned with the new resolution flow and reduces ad-hoc string handling.
…places inline scope parsing with the shared utilities and routes deletions through the extensions client lookup. The command now provides consistent progress feedback and descriptor handling, preventing accidental deletions when multiple groups share a name.
… command to the security group parent so users can discover it from the CLI. This wires the implementation into the command tree and exposes the functionality.
…ves organization and descriptor derivation to the shared helpers and starts the progress indicator earlier. This keeps the listing command consistent with the new architecture and reduces repeated code.
…s\n\nUses the shared target parsing helpers and descriptor resolution so membership listing aligns with other commands. This change ensures the command can correctly disambiguate groups across projects.
…he show subcommand with JSON and table output, leveraging the shared group finder. Users can now inspect individual security groups without manual API queries.
…ommand synopsis to the aggregated help reference so users can discover invocation details. The update mirrors the CLI behaviour introduced in this iteration.
… the group command overview with a link to the show documentation. This ensures navigation reflects the newly available command.
…e dedicated help topic describing arguments, examples, and formatting options for the show command. The page aligns with other generated command docs.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes: #78