Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove tests for removed and deprecated functionality #2612

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Oct 23, 2023

Missed this commit after quick-merging race condition between me and @mraspaud. Addition to #2610

  • Closes #xxxx
  • Tests added
  • Fully documented
  • Add your name to AUTHORS.md if not there already

@djhoese djhoese added the cleanup Code cleanup but otherwise no change in functionality label Oct 23, 2023
@djhoese djhoese self-assigned this Oct 23, 2023
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #2612 (e6261e7) into main (dc3ee7d) will increase coverage by 0.11%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2612      +/-   ##
==========================================
+ Coverage   95.07%   95.18%   +0.11%     
==========================================
  Files         354      354              
  Lines       51370    51253     -117     
==========================================
- Hits        48839    48786      -53     
+ Misses       2531     2467      -64     
Flag Coverage Δ
behaviourtests 4.24% <ø> (+<0.01%) ⬆️
unittests 95.81% <ø> (+0.11%) ⬆️

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

Files Coverage Δ
satpy/resample.py 88.93% <ø> (-0.39%) ⬇️
satpy/tests/test_resample.py 99.59% <ø> (+10.68%) ⬆️

@djhoese djhoese merged commit 93841e5 into pytroll:main Oct 23, 2023
18 of 20 checks passed
@djhoese djhoese deleted the feature-pyr2-resamplers branch October 23, 2023 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Code cleanup but otherwise no change in functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant