Skip to content

Feature/google java format 1.4#403

Closed
qweek wants to merge 4 commits into
rsocket:1.0.xfrom
qweek:feature/google-java-format-1.4
Closed

Feature/google java format 1.4#403
qweek wants to merge 4 commits into
rsocket:1.0.xfrom
qweek:feature/google-java-format-1.4

Conversation

@qweek

@qweek qweek commented Oct 3, 2017

Copy link
Copy Markdown
Contributor

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

yschimke commented Oct 3, 2017

Copy link
Copy Markdown
Member

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

qweek commented Oct 3, 2017

Copy link
Copy Markdown
Contributor Author

Let's await for review #401

@yschimke

yschimke commented Oct 5, 2017

Copy link
Copy Markdown
Member

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