You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In our Kotlin Multiplatform project we use two JVM targets (Android and desktop), therefore this plugin will use two different folders for the API dumps: api/android and api/desktop. However, in CI we disable the desktop target and only Android is present. The apiCheck then will fail, because it doesn't look in the right directory for the dump. The code is here.
To avoid the issue, either an option to always us the platform specific folder no matter how many JVM targets would work, or as an alternative an option to disable targets, e.g. we'd disable desktop and only use Android.
The text was updated successfully, but these errors were encountered:
To avoid the issue, either an option to always us the platform specific folder no matter how many JVM targets would work, or as an alternative an option to disable targets, e.g. we'd disable desktop and only use Android.
Just to explore possible solutions: would an option to use flat dump files layout (like api/<moduleName>.android.api, api/<moduleName>.desktop.api) work for you?
Yes, it would. The important piece here is that the output path and file name must stay consistent and not depend on how many JVM targets are there, so it shouldn't switch between <moduleName>.android.api and <moduleName>.api.
In our Kotlin Multiplatform project we use two JVM targets (Android and desktop), therefore this plugin will use two different folders for the API dumps:
api/android
andapi/desktop
. However, in CI we disable the desktop target and only Android is present. TheapiCheck
then will fail, because it doesn't look in the right directory for the dump. The code is here.To avoid the issue, either an option to always us the platform specific folder no matter how many JVM targets would work, or as an alternative an option to disable targets, e.g. we'd disable desktop and only use Android.
The text was updated successfully, but these errors were encountered: