Skip to content

Commit

Permalink
Automatically linting code
Browse files Browse the repository at this point in the history
  • Loading branch information
rsg-bot authored Dec 17, 2024
1 parent d74e6f8 commit 8077fa4
Show file tree
Hide file tree
Showing 9 changed files with 17 additions and 47 deletions.
1 change: 0 additions & 1 deletion .github/workflows/publish-nuget.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,3 @@ jobs:
run: |
dotnet nuget push **/*.nupkg --skip-duplicate -s nuget.org --api-key $ENV:ApiKey
dotnet nuget push **/*.snupkg --skip-duplicate -s nuget.org --api-key $ENV:ApiKey
11 changes: 9 additions & 2 deletions src/Nuke/FullToolCommandDefinition.cs
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,18 @@ string Command
)

Check warning on line 9 in src/Nuke/FullToolCommandDefinition.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/FullToolCommandDefinition.cs#L9

Added line #L9 was not covered by tests
{
[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]

/* Unmerged change from project 'Rocket.Surgery.Nuke(net9.0)'
Before:
private string DebuggerDisplay
{
get
{
return ToString();
}
}
}
}
After:
private string DebuggerDisplay => ToString();
*/
private string DebuggerDisplay => ToString();

Check warning on line 25 in src/Nuke/FullToolCommandDefinition.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/FullToolCommandDefinition.cs#L25

Added line #L25 was not covered by tests
}
8 changes: 1 addition & 7 deletions src/Nuke/GithubActions/DownloadArtifactStep.cs
Original file line number Diff line number Diff line change
Expand Up @@ -53,13 +53,7 @@ public class DownloadArtifactStep : UsingStep
public string? RunId { get; set; }

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 56 in src/Nuke/GithubActions/DownloadArtifactStep.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/DownloadArtifactStep.cs#L56

Added line #L56 was not covered by tests

Check warning on line 56 in src/Nuke/GithubActions/DownloadArtifactStep.cs

View workflow job for this annotation

GitHub Actions / lint

Possible null reference return.

/// <inheritdoc />
public override void Write(CustomFileWriter writer)
Expand Down
4 changes: 2 additions & 2 deletions src/Nuke/GithubActions/GitHubActionsStepsAttribute.cs
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ .. onePasswordServiceAccountSecrets
.Select(
z => new KeyValuePair<string, string>(
z.Name,
( string.IsNullOrWhiteSpace(z.Variable) )
( string.IsNullOrWhiteSpace(z.Variable) )

Check warning on line 176 in src/Nuke/GithubActions/GitHubActionsStepsAttribute.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/GitHubActionsStepsAttribute.cs#L176

Added line #L176 was not covered by tests
? $"{z.Path}"
: $$$"""${{ vars.{{{z.Variable}}} }}/{{{z.Path.TrimStart('/')}}}"""
)
Expand Down Expand Up @@ -234,7 +234,7 @@ .. onePasswordConnectServerSecrets
.Select(
z => new KeyValuePair<string, string>(
z.Name,
( string.IsNullOrWhiteSpace(z.Variable) )
( string.IsNullOrWhiteSpace(z.Variable) )

Check warning on line 237 in src/Nuke/GithubActions/GitHubActionsStepsAttribute.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/GitHubActionsStepsAttribute.cs#L237

Added line #L237 was not covered by tests
? $"{z.Path}"
: $$$"""${{ vars.{{{z.Variable}}} }}/{{{z.Path.TrimStart('/')}}}"""
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,13 +59,7 @@ public class RocketSurgeonGitHubActionsConfiguration : ConfigurationEntity
public GithubActionCondition? If { get; set; }

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 62 in src/Nuke/GithubActions/RocketSurgeonGitHubActionsConfiguration.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonGitHubActionsConfiguration.cs#L62

Added line #L62 was not covered by tests

/// <inheritdoc />
public override void Write(CustomFileWriter writer)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,7 @@ public class RocketSurgeonGitHubActionsVcsTrigger : GitHubActionsDetailedTrigger
public ImmutableArray<string> Types { get; set; } = [];

Check warning on line 43 in src/Nuke/GithubActions/RocketSurgeonGitHubActionsVcsTrigger.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonGitHubActionsVcsTrigger.cs#L43

Added line #L43 was not covered by tests

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 46 in src/Nuke/GithubActions/RocketSurgeonGitHubActionsVcsTrigger.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonGitHubActionsVcsTrigger.cs#L46

Added line #L46 was not covered by tests

/// <inheritdoc />
public override void Write(CustomFileWriter writer)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,7 @@ public class RocketSurgeonGitHubActionsWorkflowTrigger : GitHubActionsDetailedTr
public List<GitHubActionsWorkflowOutput> Outputs { get; set; } = [];

Check warning on line 31 in src/Nuke/GithubActions/RocketSurgeonGitHubActionsWorkflowTrigger.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonGitHubActionsWorkflowTrigger.cs#L31

Added line #L31 was not covered by tests

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 34 in src/Nuke/GithubActions/RocketSurgeonGitHubActionsWorkflowTrigger.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonGitHubActionsWorkflowTrigger.cs#L34

Added line #L34 was not covered by tests

/// <inheritdoc />
public override void Write(CustomFileWriter writer)
Expand Down
8 changes: 1 addition & 7 deletions src/Nuke/GithubActions/RocketSurgeonsGithubActionsJob.cs
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,7 @@ public class RocketSurgeonsGithubActionsJob(string name) : RocketSurgeonsGithubA
internal IDictionary<object, object> InternalData { get; } = new Dictionary<object, object>();

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 47 in src/Nuke/GithubActions/RocketSurgeonsGithubActionsJob.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/GithubActions/RocketSurgeonsGithubActionsJob.cs#L47

Added line #L47 was not covered by tests

/// <inheritdoc />
public override void Write(CustomFileWriter writer)
Expand Down
8 changes: 1 addition & 7 deletions src/Nuke/ToolDefinition.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,6 @@ public class ToolDefinition
public string[] Commands { get; set; } = [];

[System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
private string DebuggerDisplay
{
get
{
return ToString();
}
}
private string DebuggerDisplay => ToString();

Check warning on line 19 in src/Nuke/ToolDefinition.cs

View workflow job for this annotation

GitHub Actions / build

Possible null reference return.

Check warning on line 19 in src/Nuke/ToolDefinition.cs

View check run for this annotation

Codecov / codecov/patch

src/Nuke/ToolDefinition.cs#L19

Added line #L19 was not covered by tests
// ReSharper disable once NullableWarningSuppressionIsUsed
}

0 comments on commit 8077fa4

Please sign in to comment.