8000 Move to typer to manage CLI by alvarolopez · Pull Request #17 · ai4os/ai4-metadata · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Move to typer to manage CLI #17

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Aug 22, 2024
Merged

Move to typer to manage CLI #17

merged 4 commits into from
Aug 22, 2024

Conversation

alvarolopez
Copy link

No description provided.

@alvarolopez alvarolopez force-pushed the typer branch 2 times, most recently from e835ff6 to 4f0a184 Compare August 22, 2024 10:30
@alvarolopez
feat: move to Typer for CLI implementation
287577d 
We move to typer, and we implement new subcommands instead of
independent scripts:

    ai4-metadata <validate|migrate>

We keep old commands for compatiblity, but raise a deprecation warning.

Fixes #15
@alvarolopez
tests: disable some incorrect errors
3638525
@alvarolopez
feat: improve message formatting, using rich
8a63779 
Fixes #13
@alvarolopez
feat: allow loading YAML files
3f34fe8 
Also improve loading and exception management.

Fixes #12
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@alvarolopez alvarolopez merged commit 03bc86f into master Aug 22, 2024
7 checks passed
@alvarolopez alvarolopez deleted the typer branch August 22, 2024 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alvarolopez
0