Skip to content

Comments

ServiceBus: Checkstyle Part 7#3573

Merged
mssfang merged 3 commits intoAzure:masterfrom
mssfang:ServiceBus-CheckStyle-7
May 17, 2019
Merged

ServiceBus: Checkstyle Part 7#3573
mssfang merged 3 commits intoAzure:masterfrom
mssfang:ServiceBus-CheckStyle-7

Conversation

@mssfang
Copy link
Contributor

@mssfang mssfang commented May 7, 2019

PR contains only CheckStyle errors and warning fixes.

  • no new line for bracket
  • renaming private final field with capital letters
  • unable import *
  • spacing
  • using lambda function

@mssfang mssfang added Service Bus Client This issue points to a problem in the data-plane of the library. labels May 7, 2019
@mssfang mssfang requested review from conniey and yvgopal May 7, 2019 17:56
@mssfang mssfang self-assigned this May 7, 2019
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

Copy link
Member

@conniey conniey left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Member

@yvgopal yvgopal left a comment

Choose a reason for hiding this comment

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

:shipit:

@mssfang mssfang merged commit 9065284 into Azure:master May 17, 2019
@mssfang mssfang deleted the ServiceBus-CheckStyle-7 branch May 17, 2019 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. Service Bus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants