-
Notifications
You must be signed in to change notification settings - Fork 25
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 s3_utils
module
#154
Remove s3_utils
module
#154
Conversation
The JWST failure is expected until spacetelescope/jwst#7542 is merged. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #154 +/- ##
==========================================
- Coverage 65.22% 64.98% -0.25%
==========================================
Files 101 99 -2
Lines 5450 5377 -73
==========================================
- Hits 3555 3494 -61
+ Misses 1895 1883 -12
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 in Codecov by Sentry. |
3700a93
to
bf93465
Compare
bf93465
to
29b7478
Compare
looks like |
This has already been done: spacetelescope/romancal#676 |
The
s3_utils
module is no longer needed, see spacetelescope/stpipe#91 for details. This PR removes that module and cleans all the usages ofs3_utils
fromstdatamodels
.Checklist
CHANGES.rst
(either inBug Fixes
orChanges to API
)