Skip to content

Conversation

@ogenstad
Copy link
Contributor

Fixes #140

@wvandeun, did you have some other solution in mind? We could also return an error if you used this command without a name and didn't specify "--list".

@codecov
Copy link

codecov bot commented Nov 12, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/ctl/cli_commands.py 0.00% 0 Missing and 1 partial ⚠️
@@           Coverage Diff            @@
##           develop     #141   +/-   ##
========================================
  Coverage    65.40%   65.40%           
========================================
  Files           76       76           
  Lines         6923     6923           
  Branches      1367     1367           
========================================
  Hits          4528     4528           
  Misses        2026     2026           
  Partials       369      369           
Flag Coverage Δ
python-3.10 44.40% <0.00%> (ø)
python-3.11 44.40% <0.00%> (ø)
python-3.12 44.40% <0.00%> (ø)
python-3.9 43.52% <0.00%> (ø)
python-filler-3.12 23.84% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
infrahub_sdk/ctl/cli_commands.py 44.89% <0.00%> (ø)

@ogenstad ogenstad marked this pull request as ready for review November 15, 2024 16:30
@ogenstad ogenstad requested a review from a team November 15, 2024 16:30
@ogenstad ogenstad merged commit 344c8d3 into develop Nov 18, 2024
12 checks passed
@ogenstad ogenstad deleted the pog-missing-transform-name branch November 18, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: running infrahubctl transform without an argument does not list the existing transforms

4 participants