10000 Feature/google java format 1.4 by qweek · Pull Request #403 · rsocket/rsocket-java · GitHub
[go: up one dir, main page]

Skip to content

Feature/google java format 1.4 #403

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from

Conversation

qweek
Copy link
Contributor
@qweek qweek commented Oct 3, 2017

upgrade google-java-format to version 1.4

qweek added 4 commits October 3, 2017 02:45
configuration:
  explicitly set gradle version and distribution type (upgrade later)
versions:
  move dependency versions to separate file
java:
  set encoding to UTF-8
  disable specific compiler warnings (fix later)
codestyle:
  set google java format version for plugin (upgrade later)
  run formatting before compilation
benchmark:
  exclude test dependencies from jmh jar
  fix duplicate jmh generated classes (jmh-generated-classes, jmh-generated-sources)
  fix broken jmh metadata files (BenchmarkList, CompilerHints)
  fix jmh command line jar (inline dependencies with shadow plugin)
  fix jmh task for root scope (stop generate dummy jar)
  cleanup redundant out directory generated by IDEA
  remove slf4j and log4j dependencies (replace with slf4j-nop logger)
  add visual jmh report
upgrade gradle to version 4.2
change distribution type to binary
upgrade google-java-format to version 1.4
@yschimke
Copy link
Member
yschimke commented Oct 3, 2017

If you submit this at the top of the stack, I'll accept. Otherwise will wait for any feedback from Robert on the other PR first.

@qweek
Copy link
Contributor Author
qweek commented Oct 3, 2017

Let's await for review #401

@yschimke
Copy link
Member
yschimke commented Oct 5, 2017

Can you refactor without splitting up the gradle file?

@qweek qweek closed this Oct 5, 2017
@qweek qweek mentioned this pull request Oct 5, 2017
@qweek qweek deleted the feature/google-java-format-1.4 branch October 5, 2017 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0