Skip to content

ci: update precommit config#208

Merged
vincentsarago merged 2 commits intomainfrom
fix/add-top-level-all-declaration
May 5, 2026
Merged

ci: update precommit config#208
vincentsarago merged 2 commits intomainfrom
fix/add-top-level-all-declaration

Conversation

@vincentsarago
Copy link
Copy Markdown
Member

Technical Context

  • Related Issues: Fixes # (issue number)
  • Breaking Changes: [Yes/No] (If yes, please describe the impact and migration path)

Description

Provide a brief summary of the changes, the rationale behind them, and any specific areas you'd like the reviewers to focus on.


Checklist

  • Linting: Code is formatted and linted
  • Tests: Tests pass. I have included new tests for these changes where applicable.
  • Edge Cases: I have manually verified "unhappy paths" and edge cases beyond the basic success criteria (e.g., database connection timeouts, malformed input, strict mapping rejections).
  • Documentation: I have updated README.md to reflect any new environment variables, configuration changes, or breaking updates.
  • Accountability: I can explain the implementation logic for every line of code submitted.

AI tool usage

  • AI (Copilot or something similar) supported my development of this PR. See our policy about AI tool use. Use of AI tools must be indicated.

Policy: We require a "human-in-the-loop." You are the author and are fully accountable for all submitted code. Please ensure all tool-generated content is thoroughly reviewed before submission to ensure it is not an "extractive contribution" that squanders maintainer time.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 95.65%. Comparing base (d5087b1) to head (0413ea0).

Files with missing lines Patch % Lines
stac_pydantic/api/item_collection.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #208   +/-   ##
=======================================
  Coverage   95.64%   95.65%           
=======================================
  Files          25       25           
  Lines         643      644    +1     
=======================================
+ Hits          615      616    +1     
  Misses         28       28           
Flag Coverage Δ
unittests 95.65% <88.88%> (+<0.01%) ⬆️

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

☔ 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.

@vincentsarago vincentsarago merged commit 2c5989e into main May 5, 2026
7 checks passed
@vincentsarago vincentsarago deleted the fix/add-top-level-all-declaration branch May 5, 2026 07:54
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.

2 participants