Skip to content

Enable CheckStyle Plugin in Pulsar proxy - #13343

Merged
merlimat merged 1 commit into
apache:masterfrom
hezhangjian:enable-checkstyle-pulsar-proxy
Dec 16, 2021
Merged

Enable CheckStyle Plugin in Pulsar proxy#13343
merlimat merged 1 commit into
apache:masterfrom
hezhangjian:enable-checkstyle-pulsar-proxy

Conversation

@hezhangjian

Copy link
Copy Markdown
Member

Motivation

currently, the pulsar-proxy module is not protected by the checkstyle-plugin.

Modifications

  • Enable CheckStyle Plugin
  • Fix checkstyle violations

Documentation

Check the box below and label this PR (if you have committer privilege).

Need to update docs?

  • no-need-doc

java checkstyle changes, no need doc.

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Dec 15, 2021
@hezhangjian

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

@hezhangjian

Copy link
Copy Markdown
Member Author

@merlimat merlimat modified the milestones: 2.11.0, 2.10.0 Dec 16, 2021
@merlimat merlimat added the type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use label Dec 16, 2021
@merlimat
merlimat merged commit cba8800 into apache:master Dec 16, 2021
@hezhangjian
hezhangjian deleted the enable-checkstyle-pulsar-proxy branch December 16, 2021 00:39
fxbing pushed a commit to fxbing/pulsar that referenced this pull request Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants