Skip to content

Conversation

@ymh199478
Copy link
Contributor

Q A
Branch? v2.0(master)
Bug fix? No
New feature? No
Deprecations? No
Internal Changed? Yes
Removed No
Tests pass? Yes
Doc pr? No

This submission optimizes the namespace structure, rather than thinking about it in the CatLib namespace.

  • CatLib : Main frame code
  • CatLib.Container : CatLib IOC container
  • CatLib.EventDispatcher : Internal event system
  • CatLib.Support : Store some helper functions and data structures

@codecov-io
Copy link

codecov-io commented Jun 15, 2019

Codecov Report

Merging #241 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   97.63%   97.63%           
=======================================
  Files          37       37           
  Lines        2495     2495           
=======================================
  Hits         2436     2436           
  Misses         59       59
Impacted Files Coverage Δ
src/CatLib.Core/Support/Stream/SegmentStream.cs 100% <ø> (ø) ⬆️
...Lib.Core/CatLib/Events/BeforeTerminateEventArgs.cs 100% <ø> (ø) ⬆️
src/CatLib.Core/Container/Bindable.cs 98.14% <ø> (ø)
src/CatLib.Core/Container/Container.cs 99.67% <ø> (ø)
src/CatLib.Core/Support/Helper.cs 100% <ø> (ø) ⬆️
src/CatLib.Core/Support/SortSet.cs 97.13% <ø> (ø) ⬆️
src/CatLib.Core/Support/Stream/CombineStream.cs 97.4% <ø> (ø) ⬆️
.../CatLib.Core/Support/Exception/RuntimeException.cs 100% <ø> (ø) ⬆️
...tLib.Core/CatLib/Events/StartCompletedEventArgs.cs 100% <ø> (ø) ⬆️
src/CatLib.Core/Container/MethodContainer.cs 100% <ø> (ø)
... and 26 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 79bce1c...e46f021. Read the comment docs.

@ymh199478 ymh199478 merged commit c096ecb into master Jun 15, 2019
@ymh199478 ymh199478 mentioned this pull request Jun 15, 2019
55 tasks
@ymh199478 ymh199478 deleted the changed/free-namespace-ajustment branch June 15, 2019 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants