-
Notifications
You must be signed in to change notification settings - Fork 107
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
add support to other build types than only debug and release #274
Comments
ok turns out there is a workaround android {
buildTypes {
alpha {
matchingFallbacks = ['debug']
}
}
}
|
I also encountered this problem, spent a whole day on it, and even wrote a Medium article about solving it. TLDR; As it turns out, if you have a project like this: graph LR
B(base-ui) --> D("design-system<br>(with showkase)")
A(app) --> A1(feature-module) --> B
A1 --> D
you need (not entirely sure why) to specify Unfortunately, I haven’t realized that need in time and wrote almost a Gradle plugin to modify dependency metadata. Why the library has multiple versions ( |
Thanks, I do see the problem and I don't think it's intended to be like this. I think having a single artifact makes sense to me. I can fix this in the next release. |
@popovanton0 also, thank you for the write up and documenting a way to unblock people until the fix is merged |
Published a PR to fix this - #291 |
Currently only debug and release build types are supported
imagine having a setup with build types
then the dependencies won't be available in alpha builds because the way it is published to maven central
https://repo1.maven.org/maven2/com/airbnb/android/showkase/1.0.0-beta14/
compared to e.g. leakcanary
https://repo1.maven.org/maven2/com/squareup/leakcanary/leakcanary-android/2.9.1/
the problem lies with the file naming
where instead of having one artifact, the showkase library has 2: one for debug and one for release, which is not a bad idea if you want automatic switching with just adding implementation instead of having to add debugImplementation and releaseImplementation
however I'm not aware of a way to have those files added for alpha/beta/release or any other non default build type now
for this to work we would have to have two different artifacts, e.g. one for
The text was updated successfully, but these errors were encountered: