Skip to content
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

Users CLI functionality #30

Merged
merged 11 commits into from
Mar 3, 2025
Merged

Users CLI functionality #30

merged 11 commits into from
Mar 3, 2025

Conversation

beforan
Copy link
Member

@beforan beforan commented Feb 28, 2025

Pull Request Contents

♻️ Refactor
✨ Feature

Description

This PR adds a number of Users CLI actions, per #4 and the issues listed below.

It also refactors the CLI code a little, mainly to allow easy use of Spectre.Console's AnsiConsole capabilities. The main side effect is that dependeny injection service registration for the CLI is now primarily at an app level, not per command.

This simplifies some CLI command boilerplate (and injecting the AnsiConsole instances) but may later have a performance impact. But let's not prematurely optimise.

Related Issues or other material

  • This PR relates to one or more issues, detailed below

[optional] What gif best describes this PR or how it makes you feel?

linux sl command

  • I've completed all actions and tasks detailed in the PR Template

@beforan beforan merged commit a7d9c98 into main Mar 3, 2025
6 checks passed
@beforan beforan deleted the users-cli branch March 3, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants