Skip to content
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

chore(build): skip source generation in import for Jmh #494

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

ckipp01
Copy link
Contributor

@ckipp01 ckipp01 commented Aug 20, 2022

This change turns bspEnabled := false for Jmh because it involves
source generation meaning that when you do a build import it will also
do the source generation which we don't want on import.

NOTE: That if there is another release of sbt-jmh ever this is done by
default not, but not yet released: sbt/sbt-jmh#207.

Test plan

Tests should just stay green.

This change turns `bspEnabled := false` for `Jmh` because it involves
source generation meaning that when you do a build import it will also
do the source generation which we don't want on import.

NOTE: That if there is another release of sbt-jmh ever this is done by
default not, but not yet released: sbt/sbt-jmh#207.
@jdorfman jdorfman requested a review from olafurpg August 22, 2022 15:54
@olafurpg olafurpg merged commit 5a49a8b into sourcegraph:main Aug 22, 2022
@ckipp01 ckipp01 deleted the jmh branch August 22, 2022 19:48
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