Skip to content

[WIP] Address feedback on centralized config and exceptions implementation#37

Merged
SBFRF merged 1 commit into
mainfrom
copilot/sub-pr-26
Feb 2, 2026
Merged

[WIP] Address feedback on centralized config and exceptions implementation#37
SBFRF merged 1 commit into
mainfrom
copilot/sub-pr-26

Conversation

Copilot AI commented Feb 2, 2026

Copy link
Copy Markdown
Contributor
  • Initialize scene_bbox variable before try block to prevent NameError
  • Raise appropriate exception if both GeoTIFF tags and STAC bbox are unavailable
  • Add test to verify error handling for missing georeferencing data
  • Run existing tests to ensure no regressions

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Base automatically changed from refactor/phase1-foundation to main February 2, 2026 02:12
@SBFRF
SBFRF marked this pull request as ready for review February 2, 2026 02:12
Copilot AI review requested due to automatic review settings February 2, 2026 02:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SBFRF
SBFRF merged commit 33ecc14 into main Feb 2, 2026
1 check failed
@SBFRF
SBFRF deleted the copilot/sub-pr-26 branch February 2, 2026 02:13
Copilot AI requested a review from SBFRF February 2, 2026 02:13
Copilot stopped work on behalf of SBFRF due to an error February 2, 2026 02:13
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