Skip to content

Conversation

@dkomanov
Copy link
Contributor

Use reflection-based generator (RFGeneratorSource) instead of ASM-based, it does support Java8 byte-code.

Also version of JMH is upgraded.

@johnynek
Copy link
Contributor

This looks great. When CI is green, will merge.

Thank you very much! This was a blocker on 2.12 for us.

@johnynek
Copy link
Contributor

closes #295

@johnynek johnynek merged commit d9e3b0f into bazel-contrib:master Apr 1, 2018
@dkomanov
Copy link
Contributor Author

dkomanov commented Apr 1, 2018

What's about scala2_12 branch? How to merge it there?

johnynek pushed a commit that referenced this pull request Apr 1, 2018
* Upgrade versions of JMH 1.17.4 -> 1.20, asm 5.0.4 -> 6.1.1

* Add reflection-based generator for JMH (make it default)
johnynek added a commit that referenced this pull request Apr 2, 2018
Cherry pick and merge #465 into scala 2.12 branch
ianoc-stripe pushed a commit to ianoc-stripe/rules_scala that referenced this pull request Jun 12, 2018
* Upgrade versions of JMH 1.17.4 -> 1.20, asm 5.0.4 -> 6.1.1

* Add reflection-based generator for JMH (make it default)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants