Skip to content

Commit 7693c9c

Browse files
committed
Fix compiler warnings
1 parent 330fc04 commit 7693c9c

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

Libraries/src/Amazon.Lambda.Tools/DotNetCLIWrapper.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ public DotNetCLIWrapper(IToolLogger logger, string workingDirectory)
3131
/// <param name="outputLocation"></param>
3232
/// <param name="targetFramework"></param>
3333
/// <param name="configuration"></param>
34+
/// <param name="deploymentTargetPackageStoreManifestContent"></param>
3435
public int Publish(LambdaToolsDefaults defaults, string projectLocation, string outputLocation, string targetFramework, string configuration, string deploymentTargetPackageStoreManifestContent)
3536
{
3637
if (Directory.Exists(outputLocation))

Libraries/src/Amazon.Lambda.Tools/Utilities.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,7 @@ public static void ValidateMicrosoftAspNetCoreAllReference(IToolLogger logger, s
275275
var projectContent = File.ReadAllText(csprofPath);
276276

277277

278-
ValidateMicrosoftAspNetCoreAllReference(logger, manifestContent, projectContent);
278+
ValidateMicrosoftAspNetCoreAllReferenceWithManifest(logger, manifestContent, projectContent);
279279
}
280280

281281
/// <summary>
@@ -285,7 +285,7 @@ public static void ValidateMicrosoftAspNetCoreAllReference(IToolLogger logger, s
285285
/// <param name="logger"></param>
286286
/// <param name="manifestContent"></param>
287287
/// <param name="csprojContent"></param>
288-
public static void ValidateMicrosoftAspNetCoreAllReference(IToolLogger logger, string manifestContent, string csprojContent)
288+
public static void ValidateMicrosoftAspNetCoreAllReferenceWithManifest(IToolLogger logger, string manifestContent, string csprojContent)
289289
{
290290
const string ASPNET_CORE_ALL = "Microsoft.AspNetCore.All";
291291
try

Libraries/test/Amazon.Lambda.Tools.Test/ValidateAspNetCoreAllReferenceTest.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public void NewerAspNetCoreReference()
1818
var manifest = File.ReadAllText(@"ManifestTestFiles/SampleManifest.xml");
1919
var projectFile = File.ReadAllText(@"ManifestTestFiles/NewerAspNetCoreReference.xml");
2020

21-
Assert.Throws<AmazonLambdaException>(() => Utilities.ValidateMicrosoftAspNetCoreAllReference(logger, manifest, projectFile));
21+
Assert.Throws<AmazonLambdaException>(() => Utilities.ValidateMicrosoftAspNetCoreAllReferenceWithManifest(logger, manifest, projectFile));
2222
}
2323

2424
[Fact]
@@ -28,7 +28,7 @@ public void CurrentAspNetCoreReference()
2828
var manifest = File.ReadAllText(@"ManifestTestFiles/SampleManifest.xml");
2929
var projectFile = File.ReadAllText(@"ManifestTestFiles/CurrentAspNetCoreReference.xml");
3030

31-
Utilities.ValidateMicrosoftAspNetCoreAllReference(logger, manifest, projectFile);
31+
Utilities.ValidateMicrosoftAspNetCoreAllReferenceWithManifest(logger, manifest, projectFile);
3232

3333
Assert.DoesNotContain("error", logger.Buffer.ToLower());
3434
}
@@ -40,7 +40,7 @@ public void NotUsingAspNetCore()
4040
var manifest = File.ReadAllText(@"ManifestTestFiles/SampleManifest.xml");
4141
var projectFile = File.ReadAllText(@"ManifestTestFiles/CurrentAspNetCoreReference.xml");
4242

43-
Utilities.ValidateMicrosoftAspNetCoreAllReference(logger, manifest, projectFile);
43+
Utilities.ValidateMicrosoftAspNetCoreAllReferenceWithManifest(logger, manifest, projectFile);
4444

4545
Assert.DoesNotContain("error", logger.Buffer.ToLower());
4646
}

0 commit comments

Comments
 (0)