Skip to content

Conversation

@nealkruis
Copy link
Member

  • Update to new version of Courier (previously known as "Courierr")
  • Add name member to RGI for better context in messages
  • Standardize message format
  • Re-evaluate message severity

@nealkruis nealkruis added the breaking change Change that requires a major version update label Feb 6, 2024
@nealkruis nealkruis self-assigned this Feb 6, 2024
@codecov-commenter
Copy link

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (2a5926e) 86.77% compared to head (68d06f3) 92.40%.

Files Patch % Lines
src/regular-grid-interpolator-implementation.cpp 76.74% 10 Missing ⚠️
include/btwxt/grid-axis.h 66.67% 2 Missing ⚠️
src/regular-grid-interpolator-implementation.h 0.00% 2 Missing ⚠️
src/grid-axis.cpp 92.31% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
+ Coverage   86.77%   92.40%   +5.63%     
==========================================
  Files           7        6       -1     
  Lines         514      487      -27     
==========================================
+ Hits          446      450       +4     
+ Misses         68       37      -31     
Flag Coverage Δ
integration 92.40% <83.70%> (+5.63%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nealkruis nealkruis merged commit 1a8c0df into main Feb 20, 2024
@nealkruis nealkruis deleted the update-courier branch February 20, 2024 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Change that requires a major version update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants