Skip to content

Comments

ServiceBus: Checkstyle Part 10#3571

Merged
mssfang merged 5 commits intoAzure:masterfrom
mssfang:ServiceBus-CheckStyle-10
May 17, 2019
Merged

ServiceBus: Checkstyle Part 10#3571
mssfang merged 5 commits intoAzure:masterfrom
mssfang:ServiceBus-CheckStyle-10

Conversation

@mssfang
Copy link
Contributor

@mssfang mssfang commented May 7, 2019

PR only contains Checkstyle errors and warnings:
These changes are straightforward

  • no new line for bracket
  • unable import *
  • spacing

@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:50
@mssfang mssfang self-assigned this May 7, 2019
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?


@Test
public void NullBinaryDataTest() {
public void nullBinaryDataTest() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Even though the class and method are public, but it is a test class. It should not be a code-breaking change. Same to rest of methods in the class

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

@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.

I forgot about the catch {}.

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 3450f4d into Azure:master May 17, 2019
@mssfang mssfang deleted the ServiceBus-CheckStyle-10 branch May 23, 2019 00:01
azure-sdk pushed a commit to openapi-env-test/azure-sdk-for-java that referenced this pull request Aug 15, 2022
Dw/test label6 (Azure#3571)

* Adding S360 fixes already made to newer API

* Prettier run

* ss

Co-authored-by: Joseph Porter <josephporter@microsoft.com>
Co-authored-by: Wei Dong <weidong@microsoft.com>
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