Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test: fix #957. change download response url #959

Merged
merged 2 commits into from
Nov 22, 2024
Merged

test: fix #957. change download response url #959

merged 2 commits into from
Nov 22, 2024

Conversation

dfguerrerom
Copy link
Collaborator

  • It looks like planet will eventually change the previous download link... we don't have any implementation of this endpoint, we can just change the test data regression.

@codecov-commenter
Copy link

codecov-commenter commented Nov 22, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.00%. Comparing base (fa02412) to head (791ecf4).
Report is 2 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #959      +/-   ##
==========================================
- Coverage   96.02%   96.00%   -0.02%     
==========================================
  Files          40       40              
  Lines        4074     4080       +6     
==========================================
+ Hits         3912     3917       +5     
- Misses        162      163       +1     

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


🚨 Try these New Features:

@dfguerrerom dfguerrerom merged commit 3c81c9d into main Nov 22, 2024
8 checks passed
@dfguerrerom dfguerrerom deleted the planet branch November 22, 2024 11:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants