Skip to content

Commit

Permalink
Merge pull request #46 from asizikov/feature/fix-keywords-nuspec
Browse files Browse the repository at this point in the history
Remove commas from nuspec key words list, fixes #43
  • Loading branch information
asizikov authored Jan 16, 2017
2 parents 4a6501f + 707a42b commit fa6271c
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 28 deletions.
2 changes: 1 addition & 1 deletion AsyncSuffix.nuspec
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
•ReSharper 2016.3 support;
</releaseNotes>
<language>en-US</language>
<tags>async method, naming, task</tags>
<tags>async method naming task</tags>
<dependencies>
<dependency id="Wave" version="[7.0,8.0)" />
</dependencies>
Expand Down
10 changes: 2 additions & 8 deletions src/AsyncSuffix/Analyzer/ConsiderUsingAsyncSuffixHighlighting.cs
Original file line number Diff line number Diff line change
Expand Up @@ -25,20 +25,14 @@ public ConsiderUsingAsyncSuffixHighlighting(IMethodDeclaration methodDeclaration
MethodDeclaration = methodDeclaration;
}

public DocumentRange CalculateRange()
{
return MethodDeclaration.NameIdentifier.GetDocumentRange();
}
public DocumentRange CalculateRange() => MethodDeclaration.NameIdentifier.GetDocumentRange();

public string ToolTip => "Async method name does not have 'Async' suffix";

public string ErrorStripeToolTip => ToolTip;

public int NavigationOffsetPatch => 0;

public bool IsValid()
{
return MethodDeclaration == null || MethodDeclaration.IsValid();
}
public bool IsValid() => MethodDeclaration == null || MethodDeclaration.IsValid();
}
}
5 changes: 1 addition & 4 deletions src/AsyncSuffix/Analyzer/TaskExtensions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,6 @@ namespace Sizikov.AsyncSuffix.Analyzer
{
public static class TaskExtensions
{
public static bool IsTaskType(this IDeclaredType type)
{
return type.IsTask() || type.IsGenericTask();
}
public static bool IsTaskType(this IDeclaredType type) => type.IsTask() || type.IsGenericTask();
}
}
9 changes: 2 additions & 7 deletions src/AsyncSuffix/ConsiderUsingAsyncSuffixQuickFix.cs
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,8 @@ public ConsiderUsingAsyncSuffixQuickFix([NotNull] ConsiderUsingAsyncSuffixHighli
}

public IEnumerable<IntentionAction> CreateBulbItems()
{
return new[] { new ConsiderUsingAsyncSuffixBulbItem(Highlighting.MethodDeclaration) }.ToQuickFixIntentions();
}
=> new[] {new ConsiderUsingAsyncSuffixBulbItem(Highlighting.MethodDeclaration)}.ToQuickFixIntentions();

public bool IsAvailable(IUserDataHolder cache)
{
return Highlighting.IsValid();
}
public bool IsAvailable(IUserDataHolder cache) => Highlighting.IsValid();
}
}
10 changes: 2 additions & 8 deletions src/AsyncSuffix/Workflows/RenameOverloadsPageDecorator.cs
Original file line number Diff line number Diff line change
Expand Up @@ -54,15 +54,9 @@ public IRefactoringPage Commit(IProgressIndicator pi)
return irefactoringPage;
}

public bool Initialize(IProgressIndicator pi)
{
return OverloadsPage.Initialize(pi);
}
public bool Initialize(IProgressIndicator pi) => OverloadsPage.Initialize(pi);

public bool RefreshContents(IProgressIndicator pi)
{
return OverloadsPage.RefreshContents(pi);
}
public bool RefreshContents(IProgressIndicator pi) => OverloadsPage.RefreshContents(pi);

private void ShowSuggests()
{
Expand Down

0 comments on commit fa6271c

Please sign in to comment.