spacetelescope / stdatamodels

https://stdatamodels.readthedocs.io
Other
5 stars 25 forks source link

deprecate get_fileext #363

Closed braingram closed 1 week ago

braingram commented 1 week ago

As noted in #16 DataModel.get_fileext is unused.

This PR deprecates it for removal in an upcoming release.

Regression tests all pass: https://github.com/spacetelescope/RegressionTests/actions/runs/11863222622 and the logs do not contain the warning added in this PR (confirming no use in jwst).

Tasks

news fragment change types... - ``changes/.feature.rst``: new feature - ``changes/.bugfix.rst``: fixes an issue - ``changes/.doc.rst``: documentation change - ``changes/.removal.rst``: deprecation or removal of public API - ``changes/.misc.rst``: infrastructure or miscellaneous change
codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.55%. Comparing base (1e16207) to head (a70d4ea). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #363 +/- ## ========================================== + Coverage 67.52% 67.55% +0.02% ========================================== Files 114 114 Lines 5916 5917 +1 ========================================== + Hits 3995 3997 +2 + Misses 1921 1920 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features: