-
Notifications
You must be signed in to change notification settings - Fork 434
Add request count and duration telemetry #3022
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 8 commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
c5e5d91
Add request count and duration telemetry
westin-m c8bdbdf
Add request count and duration telemetry
westin-m f08864b
Merge branch 'westin/clientTelemetry' of https://github.com/AzureAD/a…
westin-m 2a739a3
Merge branch 'dev' into westin/clientTelemetry
westin-m 49790e3
small fix
westin-m ebc0740
Merge branch 'westin/clientTelemetry' of https://github.com/AzureAD/a…
westin-m 09c3b49
Feedback, remodel timing and counters
westin-m 57c1a90
Use interface for logging, move constants to class
westin-m 3262cff
some interface changes, rework tests
westin-m 4a36bdd
fix naming
westin-m 7a1847a
address interface feedback
westin-m 44b97fa
Interface and namespace changes
westin-m 7b3d16d
avoid friend assemblies
westin-m 8fd3b88
merge w main
westin-m 5e917c9
add metadata
westin-m ec46da3
Merge branch 'dev' into westin/clientTelemetry
westin-m 4011af9
doc comments for tags
westin-m 0a6fdfb
Merge branch 'westin/clientTelemetry' of https://github.com/AzureAD/a…
westin-m ded6b4c
Merge branch 'dev' into westin/clientTelemetry
GeoK 779931e
Merge branch 'dev' into westin/clientTelemetry
jennyf19 f71d13a
PR feedback
westin-m fecb571
Merge branch 'dev' into westin/clientTelemetry
westin-m 07119c1
Merge branch 'dev' into westin/clientTelemetry
westin-m a7f85cf
Apply suggestions from code review
westin-m 63c019b
change class name and add clarifying comments
westin-m 3af5ad1
Change namespace, move package reference
westin-m d3243d6
resolve merge conflicts
westin-m File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
54 changes: 54 additions & 0 deletions
54
src/Microsoft.IdentityModel.Logging/ConfigurationManagerTelemetryInstrumentation.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,54 @@ | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. | ||
|
|
||
| using System.Diagnostics; | ||
|
|
||
| namespace Microsoft.IdentityModel.Logging | ||
| { | ||
| internal class ConfigurationManagerTelemetryInstrumentation : ITelemetryInstrumentation | ||
| { | ||
| public void IncrementOperationCounter(string clientVer, string operationStatus) | ||
| { | ||
| var tagList = new TagList() | ||
| { | ||
| { TelemetryConstants.IdentityModelVersionTag, IdentityModelTelemetryUtil.ClientVer }, | ||
| { TelemetryConstants.OperationStatusTag, operationStatus } | ||
| }; | ||
|
|
||
| IdentityModelTelemetry.IncrementConfigurationManagerCounter(tagList); | ||
| } | ||
|
|
||
| public void IncrementOperationCounter(string clientVer, string operationStatus, string exceptionType) | ||
| { | ||
| var tagList = new TagList() | ||
| { | ||
| { TelemetryConstants.IdentityModelVersionTag, IdentityModelTelemetryUtil.ClientVer }, | ||
| { TelemetryConstants.OperationStatusTag, operationStatus }, | ||
| { TelemetryConstants.ExceptionTypeTag, exceptionType } | ||
| }; | ||
|
|
||
| IdentityModelTelemetry.IncrementConfigurationManagerCounter(tagList); | ||
| } | ||
|
|
||
| public void LogOperationDuration(string clientVer, long durationInMilliseconds) | ||
| { | ||
| var tagList = new TagList() | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| { | ||
| { TelemetryConstants.IdentityModelVersionTag, IdentityModelTelemetryUtil.ClientVer } | ||
| }; | ||
|
|
||
| IdentityModelTelemetry.RecordTotalDurationHistogram(durationInMilliseconds, tagList); | ||
| } | ||
|
|
||
| public void LogOperationDuration(string clientVer, long durationInMilliseconds, string exceptionType) | ||
| { | ||
| var tagList = new TagList() | ||
| { | ||
| { TelemetryConstants.IdentityModelVersionTag, IdentityModelTelemetryUtil.ClientVer }, | ||
| { TelemetryConstants.ExceptionTypeTag, exceptionType } | ||
| }; | ||
|
|
||
| IdentityModelTelemetry.RecordTotalDurationHistogram(durationInMilliseconds, tagList); | ||
| } | ||
| } | ||
| } | ||
26 changes: 26 additions & 0 deletions
26
src/Microsoft.IdentityModel.Logging/ITelemetryInstrumentation.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. | ||
|
|
||
| namespace Microsoft.IdentityModel.Logging | ||
| { | ||
| internal interface ITelemetryInstrumentation | ||
| { | ||
| internal void LogOperationDuration( | ||
| string clientVer, | ||
| long durationInMilliseconds); | ||
|
|
||
| internal void LogOperationDuration( | ||
| string clientVer, | ||
| long durationInMilliseconds, | ||
| string exceptionType); | ||
|
|
||
| internal void IncrementOperationCounter( | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| string clientVer, | ||
| string operationStatus); | ||
|
|
||
| internal void IncrementOperationCounter( | ||
| string clientVer, | ||
| string operationStatus, | ||
| string exceptionType); | ||
| } | ||
| } | ||
49 changes: 49 additions & 0 deletions
49
src/Microsoft.IdentityModel.Logging/IdentityModelTelemetry.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. | ||
|
|
||
| using System.Diagnostics; | ||
| using System.Diagnostics.Metrics; | ||
|
|
||
| namespace Microsoft.IdentityModel.Logging | ||
| { | ||
| internal class IdentityModelTelemetry | ||
| { | ||
| /// <summary> | ||
| /// Meter name for MicrosoftIdentityModel. | ||
| /// </summary> | ||
| public const string MeterName = "MicrosoftIdentityModel_Meter"; | ||
| public const string ServiceName = "MicrosoftIdentityModel"; | ||
|
|
||
| /// <summary> | ||
| /// The meter responsible for creating instruments. | ||
| /// </summary> | ||
| public static readonly Meter IdentityModelMeter = new(MeterName, "1.0.0"); | ||
|
|
||
| internal const string TotalDurationHistogramName = "IdentityModelConfigurationRequestTotalDurationInMS"; | ||
|
|
||
| /// <summary> | ||
| /// Counter to capture requests to configuration manager. | ||
| /// </summary> | ||
| internal const string IdentityModelConfigurationManagerCounterName = "IdentityModelConfigurationManagerCounter"; | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| internal const string IdentityModelConfigurationManagerCounterDescription = "Counter capturing configuration manager operations."; | ||
| internal static readonly Counter<long> ConfigurationManagerCounter = IdentityModelMeter.CreateCounter<long>(IdentityModelConfigurationManagerCounterName, description: IdentityModelConfigurationManagerCounterDescription); | ||
|
|
||
| /// <summary> | ||
| /// Histogram to capture total duration of configuration manager operations in milliseconds. | ||
| /// </summary> | ||
| internal static readonly Histogram<long> TotalDurationHistogram = IdentityModelMeter.CreateHistogram<long>( | ||
| TotalDurationHistogramName, | ||
| unit: "ms", | ||
| description: "Performance of getting configuration calls total latency"); | ||
|
|
||
| internal static void RecordTotalDurationHistogram(long requestDurationInMs, in TagList tagList) | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| { | ||
| TotalDurationHistogram.Record(requestDurationInMs, tagList); | ||
| } | ||
|
|
||
| internal static void IncrementConfigurationManagerCounter(in TagList tagList) | ||
| { | ||
| ConfigurationManagerCounter.Add(1, tagList); | ||
| } | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,48 @@ | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetry.IdentityModelConfigurationManagerCounterDescription = "Counter capturing configuration manager operations." -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetry.IdentityModelConfigurationManagerCounterName = "IdentityModelConfigurationManagerCounter" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetry.MeterName = "MicrosoftIdentityModel_Meter" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetry.ServiceName = "MicrosoftIdentityModel" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetry.TotalDurationHistogramName = "IdentityModelConfigurationRequestTotalDurationInMS" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.Automatic = "Automatic" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.ConfigurationInvalid = "ConfigurationInvalid" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.ConfigurationRetrievalFailed = "ConfigurationRetrievalFailed" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.Direct = "Direct" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.ExceptionTypeTag = "ExceptionType" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.FirstRefresh = "FirstRefresh" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.IdentityModelVersionTag = "IdentityModelVersion" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.LKG = "LastKnownGood" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.MetadataAddressTag = "MetadataAddress" -> string | ||
| const Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.OperationStatusTag = "OperationStatus" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.Automatic = "Automatic" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.ConfigurationInvalid = "ConfigurationInvalid" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.ConfigurationRetrievalFailed = "ConfigurationRetrievalFailed" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.Direct = "Direct" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.ExceptionTypeTag = "ExceptionType" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.FirstRefresh = "FirstRefresh" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.IdentityModelVersionTag = "IdentityModelVersion" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.LKG = "LastKnownGood" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.MetadataAddressTag = "MetadataAddress" -> string | ||
| const Microsoft.IdentityModel.Logging.TelemetryConstants.OperationStatusTag = "OperationStatus" -> string | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation.ConfigurationManagerTelemetryInstrumentation() -> void | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation.IncrementOperationCounter(string clientVer, string operationStatus) -> void | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation.IncrementOperationCounter(string clientVer, string operationStatus, string exceptionType) -> void | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation.LogOperationDuration(string clientVer, long durationInMilliseconds) -> void | ||
| Microsoft.IdentityModel.Logging.ConfigurationManagerTelemetryInstrumentation.LogOperationDuration(string clientVer, long durationInMilliseconds, string exceptionType) -> void | ||
| Microsoft.IdentityModel.Logging.IdentityModelTelemetry | ||
| Microsoft.IdentityModel.Logging.IdentityModelTelemetry.IdentityModelTelemetry() -> void | ||
| Microsoft.IdentityModel.Logging.ITelemetryInstrumentation | ||
| Microsoft.IdentityModel.Logging.ITelemetryInstrumentation.IncrementOperationCounter(string clientVer, string operationStatus) -> void | ||
| Microsoft.IdentityModel.Logging.ITelemetryInstrumentation.IncrementOperationCounter(string clientVer, string operationStatus, string exceptionType) -> void | ||
| Microsoft.IdentityModel.Logging.ITelemetryInstrumentation.LogOperationDuration(string clientVer, long durationInMilliseconds) -> void | ||
| Microsoft.IdentityModel.Logging.ITelemetryInstrumentation.LogOperationDuration(string clientVer, long durationInMilliseconds, string exceptionType) -> void | ||
| Microsoft.IdentityModel.Logging.TelemetryConstants | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetry.IncrementConfigurationManagerCounter(in System.Diagnostics.TagList tagList) -> void | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetry.RecordTotalDurationHistogram(long requestDurationInMs, in System.Diagnostics.TagList tagList) -> void | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.IncrementConfigurationManagerCounter(string operationStatus) -> void | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.IncrementConfigurationManagerCounter(string operationStatus, string exceptionType) -> void | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.RecordTotalDuration(long totalMilliseconds) -> void | ||
| static Microsoft.IdentityModel.Logging.IdentityModelTelemetryUtil.RecordTotalDuration(long totalMilliseconds, string exception) -> void | ||
| static readonly Microsoft.IdentityModel.Logging.IdentityModelTelemetry.ConfigurationManagerCounter -> System.Diagnostics.Metrics.Counter<long> | ||
| static readonly Microsoft.IdentityModel.Logging.IdentityModelTelemetry.IdentityModelMeter -> System.Diagnostics.Metrics.Meter | ||
| static readonly Microsoft.IdentityModel.Logging.IdentityModelTelemetry.TotalDurationHistogram -> System.Diagnostics.Metrics.Histogram<long> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,11 @@ | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. | ||
|
|
||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.JsonWebTokens, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.S2S, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.S2S.Tokens, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.S2S.Tests, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.S2S.Tokens.Tests, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.Tests, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Microsoft.IdentityModel.Protocols.OpenIdConnect.Tests, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
| [assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IdentityModel.Tokens.Jwt, PublicKey=0024000004800000940000000602000000240000525341310004000001000100b5fc90e7027f67871e773a8fde8938c81dd402ba65b9201d60593e96c492651e889cc13f1415ebb53fac1131ae0bd333c5ee6021672d9718ea31a8aebd0da0072f25d87dba6fc90ffd598ed4da35e44c398c454307e8e33b8426143daec9f596836f97c8f74750e5975c64e2189f45def46b2a2b1247adc3652bf5c308055da9")] | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. | ||
|
|
||
| namespace Microsoft.IdentityModel.Logging | ||
| { | ||
| internal static class TelemetryConstants | ||
| { | ||
| // Static attribute tags | ||
| public const string IdentityModelVersionTag = "IdentityModelVersion"; | ||
| public const string MetadataAddressTag = "MetadataAddress"; | ||
| public const string OperationStatusTag = "OperationStatus"; | ||
| public const string ExceptionTypeTag = "ExceptionType"; | ||
|
|
||
| // Configuration manager refresh statuses | ||
| public const string Automatic = "Automatic"; | ||
westin-m marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| public const string Direct = "Direct"; | ||
| public const string FirstRefresh = "FirstRefresh"; | ||
| public const string LKG = "LastKnownGood"; | ||
|
|
||
| // Configuration manager exception types | ||
| public const string ConfigurationInvalid = "ConfigurationInvalid"; | ||
| public const string ConfigurationRetrievalFailed = "ConfigurationRetrievalFailed"; | ||
| } | ||
| } | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.