Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: error #192

Closed
wants to merge 7 commits into from
Closed

Conversation

Daanvdplas
Copy link
Collaborator

@Daanvdplas Daanvdplas commented Aug 14, 2024

Closes #204

Additionally it removes the pop primitives dependency from the extension crate since it is no longer needed. This should have been added to #211 but forgotten.

Base automatically changed from daan/docs-pop_api to daan/api August 16, 2024 08:27
@Daanvdplas Daanvdplas force-pushed the daan/api branch 2 times, most recently from 5c36b4f to 6adf1b1 Compare August 16, 2024 14:26
@Daanvdplas Daanvdplas changed the title refactor: error [WIP] refactor: error Aug 17, 2024
@Daanvdplas Daanvdplas changed the base branch from daan/api to daan/tests-increase_coverage August 18, 2024 16:55
@Daanvdplas Daanvdplas changed the title [WIP] refactor: error refactor: error Aug 18, 2024
@codecov-commenter
Copy link

codecov-commenter commented Aug 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 33.24%. Comparing base (bdfdc96) to head (a5cc2c7).

@@                       Coverage Diff                        @@
##           daan/tests-increase_coverage     #192      +/-   ##
================================================================
+ Coverage                         32.76%   33.24%   +0.47%     
================================================================
  Files                                34       33       -1     
  Lines                              2994     2951      -43     
  Branches                           2994     2951      -43     
================================================================
  Hits                                981      981              
+ Misses                             1980     1937      -43     
  Partials                             33       33              
Files Coverage Δ
extension/src/lib.rs 10.59% <ø> (ø)

Copy link
Collaborator

@chungquantin chungquantin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Daanvdplas
Copy link
Collaborator Author

No longer needed due to #218

@Daanvdplas Daanvdplas closed this Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants