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

Dynamic Invocation #19

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

use make, not generate

89f32cc
Select commit
Loading
Failed to load commit list.
Open

Dynamic Invocation #19

use make, not generate
89f32cc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 17, 2025 in 0s

84.67% (+1.58%) compared to 7a9367e

View this Pull Request on Codecov

84.67% (+1.58%) compared to 7a9367e

Details

Codecov Report

Attention: Patch coverage is 86.51163% with 29 lines in your changes missing coverage. Please review.

Project coverage is 84.67%. Comparing base (7a9367e) to head (89f32cc).
Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
Sources/InterposeKit/InterposeRuntime.swift 83.78% 6 Missing ⚠️
Sources/InterposeKit/InterposeSubclass.swift 86.95% 6 Missing ⚠️
Sources/InterposeKit/ObjectHook.swift 62.50% 6 Missing ⚠️
Sources/InterposeKit/DynamicHook.swift 91.80% 5 Missing ⚠️
Sources/InterposeKit/InterposeKit.swift 62.50% 3 Missing ⚠️
Sources/SuperBuilder/src/ITKSuperBuilder.m 85.71% 2 Missing ⚠️
Sources/InterposeKit/InterposeError.swift 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   83.09%   84.67%   +1.58%     
==========================================
  Files          15       18       +3     
  Lines         899     1057     +158     
==========================================
+ Hits          747      895     +148     
- Misses        152      162      +10     

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