appsquickly / typhoon

Powerful dependency injection for Objective-C ✨✨ (https://PILGRIM.PH is the pure Swift successor to Typhoon!!)✨✨
https://pilgrim.ph
Apache License 2.0
2.7k stars 270 forks source link

#564 fix documentation warnings #569

Closed Emailrus closed 6 years ago

Emailrus commented 6 years ago

https://github.com/appsquickly/Typhoon/issues/564

codecov-io commented 6 years ago

Codecov Report

Merging #569 into master will decrease coverage by 2.13%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #569      +/-   ##
==========================================
- Coverage    89.1%   86.96%   -2.14%     
==========================================
  Files         303      295       -8     
  Lines        8653     7988     -665     
  Branches      542      542              
==========================================
- Hits         7710     6947     -763     
+ Misses        898      762     -136     
- Partials       45      279     +234
Impacted Files Coverage Δ
Tests/iOS/Typhoon-iOS/main.m 0% <0%> (-100%) :arrow_down:
...l/CircularDependencies/PrototypePropertyInjected.m 14.28% <0%> (-42.86%) :arrow_down:
Source/Factory/Internal/TyphoonAssemblyBuilder.m 26.66% <0%> (-26.67%) :arrow_down:
...ce/Factory/Internal/TyphoonMemoryManagementUtils.m 75% <0%> (-25%) :arrow_down:
...nal/TyphoonParentReferenceHydratingPostProcessor.m 25% <0%> (-25%) :arrow_down:
...hoonConfiguration/TyphoonPlistStyleConfiguration.m 65% <0%> (-20.19%) :arrow_down:
Tests/Model/CROPrototypeA.m 80% <0%> (-20%) :arrow_down:
Tests/Model/SingletonsChain/NotSingletonA.m 80% <0%> (-20%) :arrow_down:
...Model/CircularDependencies/PrototypeInitInjected.m 80% <0%> (-20%) :arrow_down:
...nConfiguration/TyphoonPropertyStyleConfiguration.m 81.25% <0%> (-18.75%) :arrow_down:
... and 138 more

Continue to review full report at Codecov.

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

jasperblues commented 6 years ago

thanks @Emailrus !