Skip to content

Conversation

Kantis
Copy link
Member

@Kantis Kantis commented Jul 4, 2025

Cherry-picked some stuff from @aSemy's work on Ks3's build setup.

This enables us to only allow publications to be created on certain targets, so we can run macOS builds and just get Apple targets artifacts, etc.

@Kantis Kantis requested a review from a team as a code owner July 4, 2025 14:05
@Kantis Kantis requested review from aSemy and removed request for a team July 4, 2025 14:05
@Kantis Kantis added this pull request to the merge queue Jul 5, 2025
Comment on lines +34 to +37
# Kotest build settings
# enabled/disable Kotlin targets, for improving local dev & CI/CD performance
kotest_enableKotlinJs=true
kotest_enableKotlinNative=false
Copy link
Contributor

@aSemy aSemy Jul 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this config also disable the native tests on CI?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch.. we should explicitly enable all targets in CI no matter the checked in config..

Merged via the queue into master with commit 3f604ef Jul 5, 2025
7 checks passed
@Kantis Kantis deleted the release-workflow branch July 5, 2025 09:29
Kantis added a commit to JesusMcCloud/kotest that referenced this pull request Jul 12, 2025
kotest#4918)

Cherry-picked some stuff from @aSemy's work on Ks3's build setup.

This enables us to only allow publications to be created on certain
targets, so we can run macOS builds and just get Apple targets
artifacts, etc.

<!-- 
If this PR updates documentation, please update all relevant versions of
the docs, see:
https://github.com/kotest/kotest/tree/master/documentation/versioned_docs
The documentation at
https://github.com/kotest/kotest/tree/master/documentation/docs is the
documentation for the next minor or major version _TO BE RELEASED_
-->
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