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
Added gradle convention plugins #3024
Conversation
|
||
kotlin { | ||
targets { | ||
js(BOTH) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We sometimes used to target js(IR)
and sometimes js(BOTH)
. Are we ready to go to IR only?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess once 1.7 comes out we could ditch BOTH ?
linuxX64() | ||
|
||
mingwX64() | ||
|
||
macosX64() | ||
macosArm64() | ||
|
||
tvos() | ||
tvosSimulatorArm64() | ||
|
||
watchosArm32() | ||
watchosArm64() | ||
watchosX86() | ||
watchosX64() | ||
watchosSimulatorArm64() | ||
|
||
iosX64() | ||
iosArm64() | ||
iosArm32() | ||
iosSimulatorArm64() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Option to configure targets, all multiplatform targets should still be default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Having a hard time figuring this one out.. perhaps hard to solve with buildSrc plugins, might be more suited for a proper internal kotest gradle plugin..
86ae7ca
to
05bc44d
Compare
This seems to have been accidentally removed in kotest#3024.
This seems to have been accidentally removed in #3024.
No description provided.