Skip to content

Commit c13c6a9

Browse files
Code cleanup
1 parent ea9ea46 commit c13c6a9

8 files changed

Lines changed: 16 additions & 7 deletions

File tree

src/Pure.DI.Core/Core/GraphWalker.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ public override int GetHashCode()
6565
{
6666
var hashCode = Target.GetHashCode();
6767
hashCode = hashCode * 397 ^ Source.GetHashCode();
68+
// ReSharper disable once ForeachCanBeConvertedToQueryUsingAnotherGetEnumerator
6869
foreach (var depIndex in DepIndices)
6970
{
7071
hashCode = hashCode * 397 ^ depIndex;

tests/Pure.DI.UsageTests/Advanced/IsLockRequiredScenario.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
// ReSharper disable UnusedParameter.Local
1717
// ReSharper disable UnusedMember.Global
1818
// ReSharper disable NotAccessedPositionalProperty.Global
19+
// ReSharper disable InvertIf
20+
// ReSharper disable UnusedMember.Local
1921
#pragma warning disable CS9113 // Parameter is unread.
2022
namespace Pure.DI.UsageTests.Advanced.IsLockRequiredScenario;
2123

tests/Pure.DI.UsageTests/Advanced/RootNameScenario.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
// ReSharper disable UnusedParameter.Local
1717
// ReSharper disable UnusedMember.Global
1818
// ReSharper disable NotAccessedPositionalProperty.Global
19+
// ReSharper disable UnusedMember.Local
1920
#pragma warning disable CS9113 // Parameter is unread.
2021
namespace Pure.DI.UsageTests.Advanced.RootNameScenario;
2122

tests/Pure.DI.UsageTests/Advanced/RootTypeScenario.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
// ReSharper disable UnusedParameter.Local
1717
// ReSharper disable UnusedMember.Global
1818
// ReSharper disable NotAccessedPositionalProperty.Global
19+
// ReSharper disable UnusedMember.Local
1920
#pragma warning disable CS9113 // Parameter is unread.
2021
namespace Pure.DI.UsageTests.Advanced.RootTypeScenario;
2122

tests/Pure.DI.UsageTests/Advanced/TagOnInjectionSiteScenario.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ class UserService(
103103

104104
public IUserRepository CloudRepository { get; } = cloudRepo;
105105

106-
public required IUserRepository BackupRepository { init; get; }
106+
public required IUserRepository BackupRepository { get; init; }
107107

108108
public IUserRepository FetcherRepository => fetcher.Repository;
109109
}

tests/Pure.DI.UsageTests/Advanced/TagOnInjectionSiteWithWildcardsScenario.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ class SmartHomeSystem(
102102

103103
public ISensor Zone2 { get; } = zone2;
104104

105-
public required ISensor OutdoorSensor { init; get; }
105+
public required ISensor OutdoorSensor { get; init; }
106106

107107
public ISensor ClimateSensor => climateControl.Sensor;
108108
}

tests/Pure.DI.UsageTests/Advanced/TagOnMemberScenario.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,6 @@ interface ICheckoutService
7373

7474
class CheckoutService : ICheckoutService
7575
{
76-
public required IPaymentGateway Gateway { init; get; }
76+
public required IPaymentGateway Gateway { get; init; }
7777
}
7878
// }

tests/Pure.DI.UsageTests/ScenarioMetadataTests.cs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// ReSharper disable CheckNamespace
22
namespace Pure.DI.UsageTests;
33

4-
public class ScenarioMetadataTests
4+
public partial class ScenarioMetadataTests
55
{
66
[Fact]
77
public void ShouldContainRequiredDocumentationMetadata()
@@ -47,12 +47,13 @@ public void ShouldContainRequiredDocumentationMetadata()
4747
errors.Add($"Suspicious encoding artifacts in {file}");
4848
}
4949

50+
// ReSharper disable once LoopCanBeConvertedToQuery
5051
foreach (var mdLink in GetMarkdownLinks(metadata))
5152
{
5253
if (mdLink.StartsWith("http://", StringComparison.OrdinalIgnoreCase) ||
5354
mdLink.StartsWith("https://", StringComparison.OrdinalIgnoreCase) ||
5455
mdLink.StartsWith("mailto:", StringComparison.OrdinalIgnoreCase) ||
55-
mdLink.StartsWith("#", StringComparison.Ordinal))
56+
mdLink.StartsWith('#'))
5657
{
5758
continue;
5859
}
@@ -86,7 +87,7 @@ private static bool TryGetMetadata(string[] lines, out IReadOnlyList<string> met
8687
}
8788

8889
metadata = block
89-
.Where(i => i.StartsWith("$", StringComparison.Ordinal))
90+
.Where(i => i.StartsWith('$'))
9091
.ToArray();
9192

9293
return true;
@@ -110,7 +111,7 @@ private static string GetSolutionDirectory()
110111

111112
private static IEnumerable<string> GetMarkdownLinks(IEnumerable<string> lines)
112113
{
113-
var regex = new System.Text.RegularExpressions.Regex(@"\[[^\]]+\]\(([^)]+)\)");
114+
var regex = MarkdownLinkRegex();
114115
foreach (var line in lines)
115116
{
116117
var matches = regex.Matches(line);
@@ -120,4 +121,7 @@ private static IEnumerable<string> GetMarkdownLinks(IEnumerable<string> lines)
120121
}
121122
}
122123
}
124+
125+
[System.Text.RegularExpressions.GeneratedRegex(@"\[[^\]]+\]\(([^)]+)\)")]
126+
private static partial System.Text.RegularExpressions.Regex MarkdownLinkRegex();
123127
}

0 commit comments

Comments
 (0)