Browse Source

minor whitespace, typo, and structured logging, changes

davidfowl/dependencies
Chad Jessup 7 years ago
committed by Ryan Nowak
parent
commit
908183290e
  1. 3
      src/Microsoft.Tye.Core/DeployServiceYamlStep.cs
  2. 2
      src/Microsoft.Tye.Core/StandardOptions.cs
  3. 2
      src/Microsoft.Tye.Hosting/TransformProjectsIntoContainers.cs
  4. 6
      src/Microsoft.Tye.Hosting/TyeHost.cs
  5. 2
      src/tye/CommonArguments.cs
  6. 2
      src/tye/Program.BuildCommand.cs

3
src/Microsoft.Tye.Core/DeployServiceYamlStep.cs

@ -71,6 +71,3 @@ namespace Microsoft.Tye
} }
} }
} }

2
src/Microsoft.Tye.Core/StandardOptions.cs

@ -185,7 +185,7 @@ namespace Microsoft.Tye
{ {
get get
{ {
return new Option(new[] { "-v", "--verbosity" }, "Output verbostiy") return new Option(new[] { "-v", "--verbosity" }, "Output verbosity")
{ {
Argument = new Argument<Verbosity>("one of: quiet|info|debug", () => Tye.Verbosity.Info) Argument = new Argument<Verbosity>("one of: quiet|info|debug", () => Tye.Verbosity.Info)
{ {

2
src/Microsoft.Tye.Hosting/TransformProjectsIntoContainers.cs

@ -23,7 +23,7 @@ namespace Microsoft.Tye.Hosting
public Task StartAsync(Model.Application application) public Task StartAsync(Model.Application application)
{ {
// This transforms a ProjectRunInfo into // This transforms a ProjectRunInfo into a container
var tasks = new List<Task>(); var tasks = new List<Task>();
foreach (var s in application.Services.Values) foreach (var s in application.Services.Values)
{ {

6
src/Microsoft.Tye.Hosting/TyeHost.cs

@ -192,13 +192,13 @@ namespace Microsoft.Tye.Hosting
else if (IsPortInUseByBinding(_application, DefaultPort)) else if (IsPortInUseByBinding(_application, DefaultPort))
{ {
// Port has been reserved for the app. // Port has been reserved for the app.
app.Logger.LogInformation($"Default dashboard port {DefaultPort} has been reserved by the application, choosing random port."); app.Logger.LogInformation("Default dashboard port {DefaultPort} has been reserved by the application, choosing random port.", DefaultPort);
return AutodetectPort; return AutodetectPort;
} }
else if (IsPortAlreadyInUse(DefaultPort)) else if (IsPortAlreadyInUse(DefaultPort))
{ {
// Port is in use by something already running. // Port is in use by something already running.
app.Logger.LogInformation($"Default dashboard port {DefaultPort} is in use, choosing random port."); app.Logger.LogInformation("Default dashboard port {DefaultPort} is in use, choosing random port.", DefaultPort);
return AutodetectPort; return AutodetectPort;
} }
else else
@ -254,7 +254,7 @@ namespace Microsoft.Tye.Hosting
new ProcessRunner(logger, ProcessRunnerOptions.FromArgs(args)), new ProcessRunner(logger, ProcessRunnerOptions.FromArgs(args)),
}; };
// If the docker command is specified then transport the ProjectRunInfo into DockerRunInfo // If the docker command is specified then transform the ProjectRunInfo into DockerRunInfo
if (args.Contains("--docker")) if (args.Contains("--docker"))
{ {
processors.Insert(0, new TransformProjectsIntoContainers(logger)); processors.Insert(0, new TransformProjectsIntoContainers(logger));

2
src/tye/CommonArguments.cs

@ -54,7 +54,7 @@ namespace Microsoft.Tye
if (files.Length > 1) if (files.Length > 1)
{ {
errorMessage = $"More than one matching files was found in directory '{directoryPath}'."; errorMessage = $"More than one matching file was found in directory '{directoryPath}'.";
filePath = default; filePath = default;
return false; return false;
} }

2
src/tye/Program.BuildCommand.cs

@ -16,7 +16,7 @@ namespace Microsoft.Tye
{ {
public static Command CreateBuildCommand() public static Command CreateBuildCommand()
{ {
var command = new Command("build", "build container for the application") var command = new Command("build", "build containers for the application")
{ {
CommonArguments.Path_Required, CommonArguments.Path_Required,
StandardOptions.Interactive, StandardOptions.Interactive,

Loading…
Cancel
Save