Skip to content

Conversation

@gadomski
Copy link
Member

@gadomski gadomski commented May 13, 2021

Related Issue(s): #51

Description: Checks to see if there's no command and, if so, prints usage. I don't error (contra the original issue) because I don't feel like a naked invocation is an error, but if others feel differently I'm happy to change.

PR Checklist:

  • Code is formatted
  • Tests pass
  • Changes are added to the CHANGELOG

@codecov-commenter
Copy link

codecov-commenter commented May 13, 2021

Codecov Report

Merging #52 (f5f7c88) into main (ee54486) will increase coverage by 0.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #52      +/-   ##
==========================================
+ Coverage   85.23%   85.35%   +0.11%     
==========================================
  Files          13       13              
  Lines         630      635       +5     
==========================================
+ Hits          537      542       +5     
  Misses         93       93              
Impacted Files Coverage Δ
pystac_client/cli.py 66.66% <100.00%> (+1.96%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee54486...f5f7c88. Read the comment docs.

@matthewhanson matthewhanson merged commit 68db8e2 into stac-utils:main May 14, 2021
@gadomski gadomski deleted the fix/issues/51-cli-no-arguments branch May 14, 2021 21:00
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.

3 participants