Minor

DevTeam · Nov 29, 2024 · 6b202cd · 6b202cd
1 parent 053fddc
commit 6b202cd
Show file tree

Hide file tree

Showing 4 changed files with 77 additions and 21 deletions.
diff --git a/README.md b/README.md
@@ -102,14 +102,6 @@ DI.Setup(nameof(Composition))
 > [!NOTE]
 > In fact, the `Bind().As(Singleton).To<Random>()` binding is unnecessary since _Pure.DI_ supports many .NET BCL types out of the box, including [Random](https://github.com/DevTeam/Pure.DI/blob/27a1ccd604b2fdd55f6bfec01c24c86428ddfdcb/src/Pure.DI.Core/Features/Default.g.cs#L289). It was added just for the example of using the _Singleton_ lifetime.
 
-The Pure.DI source code generator works like this:
-
-```mermaid
-flowchart TD
-    setups[DI setups analysis] --> types["`Types analysis
-    constructors/methods/properties/fields`"] --> deps[Creating a dependency graph] --> verification[Dependency graph verification] --> code[Code generation]
-```
-
 The above code specifies the generation of a partial class named *__Composition__*, this name is defined in the `DI.Setup(nameof(Composition))` call. This class contains a *__Root__* property that returns a graph of objects with an object of type *__Program__* as the root. The type and name of the property is defined by calling `Root<Program>("Root")`. The code of the generated class looks as follows:
 
 ```c#
@@ -367,13 +359,13 @@ partial class Composition
 }
 ```
 
-</details>
-
 The _compositionTypeName_ parameter can be omitted
 
 - if the setup is performed inside a partial class, then the composition will be created for this partial class
 - for the case of a class with composition kind `CompositionKind.Global`, see [this example](readme/global-compositions.md)
 
+</details>
+
 <details>
 <summary>Setup arguments</summary>
 
@@ -860,6 +852,42 @@ Then documentation for the composition root:
 
 </details>
 
+<details>
+<summary>Code generation workflow</summary>
+
+```mermaid
+flowchart TD
+    start@{ shape: circle, label: "Start" }
+    setups[fa:fa-search DI setups analysis]
+    types["`fa:fa-search Types analysis
+    constructors/methods/properties/fields`"] 
+    subgraph dep[Dependency graph]
+    option[fa:fa-search Selecting a next dependency set]
+    creating[fa:fa-cog Creating a dependency graph variant]
+    verification{fa:fa-check-circle Verification}
+    end
+    codeGeneration[fa:fa-code Code generation]
+    compilation[fa:fa-cog Compilation]
+    failed@{ shape: dbl-circ, label: "fa:fa-thumbs-down Compilation failed" }
+    success@{ shape: dbl-circ, label: "fa:fa-thumbs-up Success" }
+
+    start ==> setups
+    setups -.->|Has problems| failed
+    setups ==> types
+    types -.-> |Has problems| failed
+    types ==> option
+    option ==> creating
+    option -.-> |There are no other options| failed
+    creating ==> verification
+    verification -->|Has problems| option
+    verification ==>|Correct| codeGeneration
+    codeGeneration ==> compilation
+    compilation -.-> |Has problems| failed
+    compilation ==> success
+```
+
+</details>
+
 ## NuGet packages
 
 |                      |                                                                                                                     |                                                            |

diff --git a/readme/FooterTemplate.md b/readme/FooterTemplate.md
@@ -62,13 +62,13 @@ partial class Composition
 }
 ```
 
-</details>
-
 The _compositionTypeName_ parameter can be omitted
 
 - if the setup is performed inside a partial class, then the composition will be created for this partial class
 - for the case of a class with composition kind `CompositionKind.Global`, see [this example](readme/global-compositions.md)
 
+</details>
+
 <details>
 <summary>Setup arguments</summary>
 
@@ -555,6 +555,42 @@ Then documentation for the composition root:
 
 </details>
 
+<details>
+<summary>Code generation workflow</summary>
+
+```mermaid
+flowchart TD
+    start@{ shape: circle, label: "Start" }
+    setups[fa:fa-search DI setups analysis]
+    types["`fa:fa-search Types analysis
+    constructors/methods/properties/fields`"] 
+    subgraph dep[Dependency graph]
+    option[fa:fa-search Selecting a next dependency set]
+    creating[fa:fa-cog Creating a dependency graph variant]
+    verification{fa:fa-check-circle Verification}
+    end
+    codeGeneration[fa:fa-code Code generation]
+    compilation[fa:fa-cog Compilation]
+    failed@{ shape: dbl-circ, label: "fa:fa-thumbs-down Compilation failed" }
+    success@{ shape: dbl-circ, label: "fa:fa-thumbs-up Success" }
+
+    start ==> setups
+    setups -.->|Has problems| failed
+    setups ==> types
+    types -.-> |Has problems| failed
+    types ==> option
+    option ==> creating
+    option -.-> |There are no other options| failed
+    creating ==> verification
+    verification -->|Has problems| option
+    verification ==>|Correct| codeGeneration
+    codeGeneration ==> compilation
+    compilation -.-> |Has problems| failed
+    compilation ==> success
+```
+
+</details>
+
 ## NuGet packages
 
 |                      |                                                                                                                     |                                                            |

diff --git a/readme/ReadmeTemplate.md b/readme/ReadmeTemplate.md
@@ -62,14 +62,6 @@ DI.Setup(nameof(Composition))
 > [!NOTE]
 > In fact, the `Bind().As(Singleton).To<Random>()` binding is unnecessary since _Pure.DI_ supports many .NET BCL types out of the box, including [Random](https://github.com/DevTeam/Pure.DI/blob/27a1ccd604b2fdd55f6bfec01c24c86428ddfdcb/src/Pure.DI.Core/Features/Default.g.cs#L289). It was added just for the example of using the _Singleton_ lifetime.
 
-The Pure.DI source code generator works like this:
-
-```mermaid
-flowchart TD
-    setups[DI setups analysis] --> types["`Types analysis
-    constructors/methods/properties/fields`"] --> deps[Creating a dependency graph] --> verification[Dependency graph verification] --> code[Code generation]
-```
-
 The above code specifies the generation of a partial class named *__Composition__*, this name is defined in the `DI.Setup(nameof(Composition))` call. This class contains a *__Root__* property that returns a graph of objects with an object of type *__Program__* as the root. The type and name of the property is defined by calling `Root<Program>("Root")`. The code of the generated class looks as follows:
 
 ```c#

diff --git a/src/Pure.DI.Core/Features/Default.g.cs b/src/Pure.DI.Core/Features/Default.g.cs
@@ -111,9 +111,9 @@ private static void Setup()
                     .To(_ => global::System.Buffers.ArrayPool<TT>.Shared)
 #endif
                 .Bind<global::System.Collections.Generic.IList<TT>>()
+                .Bind<global::System.Collections.Generic.ICollection<TT>>()
                 .Bind<global::System.Collections.Generic.List<TT>>()
                     .To((TT[] arr) => new global::System.Collections.Generic.List<TT>(arr))
-                .Bind<global::System.Collections.Generic.ICollection<TT>>()
                 .Bind<global::System.Collections.ObjectModel.Collection<TT>>()
                     .To((TT[] arr) => new global::System.Collections.ObjectModel.Collection<TT>(arr))
 #if NETSTANDARD || NET || NETCOREAPP || NET45_OR_GREATER