Skip to content

Update Item Assets Extension#564

Merged
duckontheweb merged 5 commits into
stac-utils:mainfrom
duckontheweb:update/item-assets-extension
Jul 17, 2021
Merged

Update Item Assets Extension#564
duckontheweb merged 5 commits into
stac-utils:mainfrom
duckontheweb:update/item-assets-extension

Conversation

@duckontheweb
Copy link
Copy Markdown
Contributor

@duckontheweb duckontheweb commented Jul 17, 2021

Related Issue(s):

Description:

Updates docstrings and adds tests for the Item Assets Extension.

PR Checklist:

  • Code is formatted (run pre-commit run --all-files)
  • Tests pass (run scripts/test)
  • Documentation has been updated to reflect changes, if applicable
  • This PR maintains or improves overall codebase code coverage.
  • Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

@duckontheweb duckontheweb requested a review from lossyrob July 17, 2021 03:01
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 17, 2021

Codecov Report

❌ Patch coverage is 95.83333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.69%. Comparing base (e45af8e) to head (bb5042c).
⚠️ Report is 865 commits behind head on main.

Files with missing lines Patch % Lines
pystac/extensions/item_assets.py 71.42% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #564      +/-   ##
==========================================
+ Coverage   94.54%   94.69%   +0.14%     
==========================================
  Files          71       72       +1     
  Lines       10707    10755      +48     
  Branches     1067     1068       +1     
==========================================
+ Hits        10123    10184      +61     
+ Misses        412      395      -17     
- Partials      172      176       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@duckontheweb duckontheweb added this to the 1.0.0 milestone Jul 17, 2021
@duckontheweb duckontheweb merged commit 3dff1b6 into stac-utils:main Jul 17, 2021
@duckontheweb duckontheweb deleted the update/item-assets-extension branch July 17, 2021 15:57
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.

Testing and documentation for item-assets extension

3 participants