Skip to content

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

Merged
olafurpg merged 1 commit intosourcegraph:mainfrom
ckipp01:jmh
Aug 22, 2022
Merged

chore(build): skip source generation in import for Jmh#494
olafurpg merged 1 commit intosourcegraph:mainfrom
ckipp01:jmh

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