CCSDSPy / ccsdspy

I/O interface and utilities for CCSDS binary spacecraft data in Python. Library used in flight missions at NASA, NOAA, and SWRI
https://ccsdspy.org
BSD 3-Clause "New" or "Revised" License
74 stars 18 forks source link

Spelling and missing paren #84

Closed gfireman closed 11 months ago

gfireman commented 11 months ago

Fixed a missing paren in PacketField.repr().

While I was at it, fixed some obviously misspelled words. Weird thing is that the following "words" keep reappearing in cleanly-built html, even though they don't appear anywhere in the source files:

converteed indeces packats replacment Retreive suports

codecov[bot] commented 11 months ago

Codecov Report

Merging #84 (ccce942) into main (303331c) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #84   +/-   ##
=======================================
  Coverage   95.87%   95.87%           
=======================================
  Files           7        7           
  Lines         654      654           
=======================================
  Hits          627      627           
  Misses         27       27           
Files Changed Coverage Δ
ccsdspy/converters.py 100.00% <ø> (ø)
ccsdspy/decode.py 95.83% <ø> (ø)
ccsdspy/packet_fields.py 96.66% <ø> (ø)
ccsdspy/utils.py 94.20% <ø> (ø)
ccsdspy/packet_types.py 95.26% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

ddasilva commented 11 months ago

Thanks so much! I am happy to merge this.

Weird thing is that the following "words" keep reappearing in cleanly-built html, even though they don't appear anywhere in the source files:

The docs pick up on the installed version of the module (in site-packages), not the version of the code in the directory above. Easiest solution is to just run pip install . in the directory above before you build the docs. Let me know if this doesn't work for you.

gfireman commented 11 months ago

That works - thanks!