Skip to content

Add novas_str_degrees() and novas_str_hours()#135

Merged
attipaci merged 3 commits into
mainfrom
str-degrees-hours
Mar 4, 2025
Merged

Add novas_str_degrees() and novas_str_hours()#135
attipaci merged 3 commits into
mainfrom
str-degrees-hours

Conversation

@attipaci

@attipaci attipaci commented Mar 2, 2025

Copy link
Copy Markdown
Collaborator
  • Add novas_str_degrees() and novas_str_hours()
  • novas_dms_degrees() / novas_hms_hours() to set errno to EINVAL if there are unparsed tokens.
  • Allow ommitting the seconds component in HMS/DMS specifications.
  • Some doxygen apidoc corrections.
  • Allow 2-componend h+m or d+m parsing
  • Allow punctuation after compass direction

@attipaci attipaci added the enhancement New feature or request label Mar 2, 2025
@attipaci attipaci added this to the 1.3.0 milestone Mar 2, 2025
@attipaci attipaci self-assigned this Mar 2, 2025
@codecov

codecov Bot commented Mar 2, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.55%. Comparing base (55f87aa) to head (5b1adcf).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #135   +/-   ##
=======================================
  Coverage   99.55%   99.55%           
=======================================
  Files           9        9           
  Lines        4036     4070   +34     
  Branches      735      748   +13     
=======================================
+ Hits         4018     4052   +34     
  Misses          4        4           
  Partials       14       14           
Flag Coverage Δ
unittests 99.55% <100.00%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
src/parse.c 100.00% <100.00%> (ø)
src/timescale.c 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4582b51...5b1adcf. Read the comment docs.

@attipaci attipaci force-pushed the str-degrees-hours branch 3 times, most recently from 07f8f98 to bd9d0b7 Compare March 4, 2025 08:26
@attipaci attipaci force-pushed the str-degrees-hours branch from bd9d0b7 to 0f18e63 Compare March 4, 2025 08:32
@attipaci attipaci marked this pull request as ready for review March 4, 2025 08:33
@attipaci attipaci merged commit ff4ec52 into main Mar 4, 2025
@attipaci attipaci deleted the str-degrees-hours branch March 4, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant