search

feder-observatory / stellarphot

Stellar aperture photometry
https://stellarphot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
4 stars 12 forks source link

Fixed spellings in autoui_widget_viewer.upynb and stellarphot/core.py #367

Closed JuanCab closed 4 months ago

JuanCab commented 4 months ago

This PR fixes a bunch of typos (and in some cases, troublesome variable names that are confused for typos) that were caught by codespell. This should reduce the number of failed codespell checks dramatically. :).

I also updated ignore-words.txt to not flag some words we legitimately misspell. :)

mwcraig commented 4 months ago

@JuanCab -- can you try rebasing this to remove the conflict? Not sure why none of the tests are running either...

mwcraig commented 4 months ago

Looks like there is still one spelling error....

JuanCab commented 4 months ago

Not sure why that happened, but things are fixed and I suspect this will will now pass the tests. In the process of fixing the merge, I discovered one file failing the codespell check, so that has been fixed as well.

JuanCab commented 4 months ago

Looks like there is still one spelling error....

Fixed now with commit 290e646

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 76.48%. Comparing base (63b1fe9) to head (290e646). Report is 255 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #367 +/- ## ======================================= Coverage 76.48% 76.48% ======================================= Files 27 27 Lines 3665 3665 ======================================= Hits 2803 2803 Misses 862 862 ```

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