Skip to content

Commit c4590be

Browse files
authored
make dotnet_style_require_accessibility_modifiers a warning (#1269)
1 parent 221dddd commit c4590be

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

.editorconfig

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,9 @@ dotnet_style_coalesce_expression = true : suggestion
5757
dotnet_style_null_propagation = true : suggestion
5858
dotnet_style_explicit_tuple_names = true : suggestion
5959

60+
# Require default accessibility modifiers to be defined
61+
dotnet_style_require_accessibility_modifiers=always:warning
62+
6063
# Naming rules - async methods to be prefixed with Async
6164
dotnet_naming_rule.async_methods_must_end_with_async.severity = warning
6265
dotnet_naming_rule.async_methods_must_end_with_async.symbols = method_symbols

src/Sentry/Envelopes/Envelope.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,27 +72,27 @@ public async Task SerializeAsync(Stream stream, CancellationToken cancellationTo
7272
public void Dispose() => Items.DisposeAll();
7373

7474
// limited SDK information (no packages)
75-
static IReadOnlyDictionary<string, string?> sdkHeader = new Dictionary<string, string?>(2, StringComparer.Ordinal)
75+
private static readonly IReadOnlyDictionary<string, string?> SdkHeader = new Dictionary<string, string?>(2, StringComparer.Ordinal)
7676
{
7777
["name"] = SdkVersion.Instance.Name,
7878
["version"] = SdkVersion.Instance.Version
7979
};
8080

81-
static IReadOnlyDictionary<string, object?> defaultHeader = new Dictionary<string, object?>(1, StringComparer.Ordinal)
81+
private static readonly IReadOnlyDictionary<string, object?> DefaultHeader = new Dictionary<string, object?>(1, StringComparer.Ordinal)
8282
{
83-
["sdk"] = sdkHeader
83+
["sdk"] = SdkHeader
8484
};
8585

8686
private static IReadOnlyDictionary<string, object?> CreateHeader(SentryId? eventId = null)
8787
{
8888
if (eventId is null)
8989
{
90-
return defaultHeader;
90+
return DefaultHeader;
9191
}
9292

9393
return new Dictionary<string, object?>(2, StringComparer.Ordinal)
9494
{
95-
["sdk"] = sdkHeader,
95+
["sdk"] = SdkHeader,
9696
[EventIdKey] = eventId.Value.ToString()
9797
};
9898
}

0 commit comments

Comments
 (0)