Skip to content

Commit

Permalink
Fixed double parantheses (as proposed by toughengineer thanks :))
Browse files Browse the repository at this point in the history
  • Loading branch information
StefanKert committed Jul 5, 2019
1 parent 8fd3995 commit 10838b8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
13 changes: 6 additions & 7 deletions src/BuildVision.UI/Helpers/BuildMessagesFactory.cs
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ private string GetTimeString(DateTime? startTime)
}
}

private string GetBeginAtString(BuildAction? buildAction)
private static string GetBeginAtString(BuildAction? buildAction)
{
switch (buildAction.Value)
{
Expand All @@ -75,7 +75,7 @@ private string GetBeginAtString(BuildAction? buildAction)
}
}

private string GetActionName(BuildAction buildAction)
private static string GetActionName(BuildAction buildAction)
{
switch (buildAction)
{
Expand All @@ -92,7 +92,7 @@ private string GetActionName(BuildAction buildAction)
}
}

private string GetUnitName(BuildScope buildScope)
private static string GetUnitName(BuildScope buildScope)
{
string unitName = "";
switch (buildScope)
Expand Down Expand Up @@ -210,7 +210,7 @@ private string GetBuildDoneMajorMessage(IBuildInformationModel buildInformationM
return resultMainString;
}

private string GetResultName(BuildResultState resultState)
private static string GetResultName(BuildResultState resultState)
{
switch (resultState)
{
Expand Down Expand Up @@ -243,9 +243,8 @@ private string GetBuildDoneExtraMessage(IBuildInformationModel buildInformationM
return string.Empty;
}

TimeSpan timeSpan = buildInformationModel.BuildFinishTime.Value.Subtract(buildInformationModel.BuildStartTime.Value);
string extraTimePartString = GetExtraTimePartString(timeSpan);
return string.Format(_packageSettingsProvider.Settings.BuildMessagesSettings.ExtraMessageStringFormat, extraTimePartString);
var timeSpan = buildInformationModel.BuildFinishTime.Value.Subtract(buildInformationModel.BuildStartTime.Value);
return GetExtraTimePartString(timeSpan);
}

private string GetExtraTimePartString(TimeSpan timeSpan)
Expand Down
2 changes: 1 addition & 1 deletion src/BuildVision/Services/BuildInformationProvider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public class BuildInformationProvider : IBuildInformationProvider

public IBuildInformationModel BuildInformationModel { get; } = new BuildInformationModel();
public ObservableCollection<IProjectItem> Projects { get; } = new ObservableRangeCollection<IProjectItem>();

[ImportingConstructor]
public BuildInformationProvider(
[Import(typeof(IBuildOutputLogger))] IBuildOutputLogger buildOutputLogger,
Expand Down

1 comment on commit 10838b8

@yannduran
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'Fixed double parentheses (as proposed by toughengineer thanks :))'

it was me! ME I say!!! hehehehehehe

Please sign in to comment.