Skip to content

Move TemplateError to errors.py#1018

Merged
gadomski merged 3 commits into
stac-utils:mainfrom
pjhartzell:issues/686-template-error
Mar 1, 2023
Merged

Move TemplateError to errors.py#1018
gadomski merged 3 commits into
stac-utils:mainfrom
pjhartzell:issues/686-template-error

Conversation

@pjhartzell
Copy link
Copy Markdown
Collaborator

@pjhartzell pjhartzell commented Feb 28, 2023

Related Issue(s):

Description:

Duplicates TemplateError from layout.py to errors.py and updates all references to point to the definition in errors.py. All custom exceptions are now in errors.py. Per #685, the existing TemplateError remains in layout.py but is now marked as deprecated and noted that it will be removed in the next major release (2.0.0).

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.

@pjhartzell pjhartzell requested a review from gadomski February 28, 2023 21:35
Comment thread pystac/layout.py Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 28, 2023

Codecov Report

Patch coverage: 63.63% and project coverage change: -0.03 ⚠️

Comparison is base (4c5f1d1) 90.32% compared to head (24d7ccd) 90.30%.

❗ Current head 24d7ccd differs from pull request most recent head beb882c. Consider uploading reports for the commit beb882c to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1018      +/-   ##
==========================================
- Coverage   90.32%   90.30%   -0.03%     
==========================================
  Files          47       47              
  Lines        6184     6189       +5     
  Branches      929      929              
==========================================
+ Hits         5586     5589       +3     
- Misses        420      422       +2     
  Partials      178      178              
Impacted Files Coverage Δ
pystac/__init__.py 94.00% <ø> (ø)
pystac/layout.py 87.14% <55.55%> (-0.78%) ⬇️
pystac/errors.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pjhartzell pjhartzell requested a review from gadomski March 1, 2023 15:16
Copy link
Copy Markdown
Member

@gadomski gadomski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good ... I'm going to rebase and squash the history a bit.

@gadomski gadomski force-pushed the issues/686-template-error branch from 24d7ccd to beb882c Compare March 1, 2023 18:38
@gadomski gadomski enabled auto-merge March 1, 2023 18:38
@gadomski gadomski added this to the 1.7 milestone Mar 1, 2023
@gadomski gadomski added this pull request to the merge queue Mar 1, 2023
Merged via the queue into stac-utils:main with commit 1c49441 Mar 1, 2023
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.

Move TemplateError to errors module

3 participants