Skip to content

Commit

Permalink
Restore language version 10
Browse files Browse the repository at this point in the history
  • Loading branch information
wkirschenmann committed Aug 23, 2023
1 parent d6edac0 commit f694288
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
7 changes: 2 additions & 5 deletions Client/src/Common/Submitter/ApiExt/GrpcArmoniKClient.cs
Original file line number Diff line number Diff line change
Expand Up @@ -410,11 +410,8 @@ internal static void ValidateRetryArguments(int maxRetries,
if (maxRetries != 1 || totalTimeoutMs != 1e10)
{
throw new ArgumentOutOfRangeException(nameof(maxRetries),
$"""
{nameof(GrpcArmoniKClient)} has no retry policy and only support a single call.
{maxRetries} must be equal to 1.
{totalTimeoutMs} must be equal to 1e10 ({1e10}).
""");
$"{nameof(GrpcArmoniKClient)} has no retry policy and only support a single call.\r\n" +
$"{maxRetries} must be equal to 1.\r\n" + $"{totalTimeoutMs} must be equal to 1e10 ({1e10}).");
}
}
}
6 changes: 1 addition & 5 deletions Client/src/Common/Submitter/ApiExt/RetryArmoniKClient.cs
Original file line number Diff line number Diff line change
Expand Up @@ -244,11 +244,7 @@ internal async Task<T> ApplyRetryPolicy<T>(Func<CancellationToken, Task<T>> acti

if (policyResult.Outcome == OutcomeType.Failure)
{
throw new ArmoniKException($"""
Call to {callerName} failed the retry policy.
Reason is: {policyResult.FaultType}.
See previous log for details.
""",
throw new ArmoniKException($"Call to {callerName} failed the retry policy.\r\nReason is: {policyResult.FaultType}.\r\nSee previous log for details.",
policyResult.FinalException);
}

Expand Down
2 changes: 1 addition & 1 deletion Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@
<!--Code analysis-->
<EnableNETAnalyzers>true</EnableNETAnalyzers>
<Nullable>enable</Nullable>
<LangVersion>11</LangVersion>
<LangVersion>10</LangVersion>
<!-- Ready for the future
<AnalysisLevel>latest</AnalysisLevel>
<AnalysisMode>AllEnabledByDefault</AnalysisMode>
Expand Down

0 comments on commit f694288

Please sign in to comment.