Skip to content

Conversation

skydoves
Copy link
Member

@skydoves skydoves commented Apr 4, 2025

Motivation

@skydoves skydoves requested a review from a team April 4, 2025 00:26
@skydoves skydoves self-assigned this Apr 4, 2025
Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

:shipit:

library.gradle Outdated
@@ -25,12 +25,6 @@ android {
}
}

project.afterEvaluate {
// Remove afterEvaluate
// after https://github.com/Kotlin/kotlinx-kover/issues/362 is fixed
Copy link
Contributor

Choose a reason for hiding this comment

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

🙌

Copy link
Member

Choose a reason for hiding this comment

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

Kover is now failing. Maybe this was needed after all?

Copy link
Member

@JayShortway JayShortway left a comment

Choose a reason for hiding this comment

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

Let's go! 🙌

@skydoves skydoves enabled auto-merge April 4, 2025 11:49
github-merge-queue bot pushed a commit that referenced this pull request Apr 7, 2025
### Motivation

As we've discussed, making purchases module to transitive dependency for
the `ui` module. I'm going to apply for the `amazon` module as well once
#2327 is merged.
@skydoves
Copy link
Member Author

@tonidero @JayShortway Do you have any ideas for passing the test workflow? 😄

@JayShortway
Copy link
Member

Ugh, those Amazon tests are now failing with

Expected stackmap frame at this location.

I don't know what causes it, but I've seen it reported before: RevenueCat/purchases-kmp#271

I think it's related to the Amazon dependency and JDK version, but I don't know why it's suddenly an issue. Maybe updating the dependency from 3.0.5 to 3.0.7 helps? (Although there's nothing mentioned in the changelog.)

Otherwise we can try setting -noverify in the :feature:amazon module, as suggested here.

@JayShortway
Copy link
Member

@skydoves @tonidero This fixes Amazon tests for me locally: 296e40e.

Copy link
Member

@JayShortway JayShortway left a comment

Choose a reason for hiding this comment

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

Almost there! 😄

@skydoves
Copy link
Member Author

@JayShortway This (296e40e) fixed on my local device but it stills failed on CI 😞

@JayShortway
Copy link
Member

@skydoves Hmm I don't think it's Amazon anymore. The failures are different this time.

  • test_cec_debug and test_cec_release are variant incompatibilities. Maybe matchingFallbacks can help here too?
  • test_defaults_release seems to be Kover related, see this related comment.

Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.32%. Comparing base (579a2b9) to head (b6e7401).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2327   +/-   ##
=======================================
  Coverage   78.32%   78.32%           
=======================================
  Files         300      300           
  Lines       11126    11126           
  Branches     1551     1551           
=======================================
  Hits         8714     8714           
  Misses       1732     1732           
  Partials      680      680           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@skydoves skydoves added this pull request to the merge queue Aug 19, 2025
@skydoves
Copy link
Member Author

@JayShortway @tonidero Eventually it has passed! 🚀 Thank you so much for your support and patience!

Merged via the queue into main with commit b31b0d8 Aug 19, 2025
20 checks passed
@skydoves skydoves deleted the kts/migration-1 branch August 19, 2025 03:10
@JayShortway
Copy link
Member

@skydoves Awesome, thanks for persisting! 😄 Glad to get this in.

This was referenced Aug 20, 2025
tonidero pushed a commit that referenced this pull request Aug 21, 2025
**This is an automatic release.**

## RevenueCat SDK
### 🐞 Bugfixes
* Use `Block store` to backup anonymous user ids across installations
(#2595) via Toni Rico (@tonidero)

## RevenueCatUI SDK
### Paywallv2
#### 🐞 Bugfixes
* Fixes price formatting discrepancies on Paywalls for `{{
product.price_per_[day|week|month|year] }}` (#2604) via JayShortway
(@JayShortway)

### 🔄 Other Changes
* Revert dokka 2 and gradle 9 update (#2618) via Toni Rico (@tonidero)
* Introduce runtime annotations library and add stability annotations
for increasing UI performances (#2608) via Jaewoong Eum (@skydoves)
* Override presented offering context paywalls without offering (#2612)
via Toni Rico (@tonidero)
* Add APIs for hybrid SDKs to set presentedOfferingContext (#2610) via
Toni Rico (@tonidero)
* Bump Baseline Profiles to 1.4.0 and update profiles (#2611) via
Jaewoong Eum (@skydoves)
* Migrate deprecated kotlinOptions to compilerOptions (#2607) via
Jaewoong Eum (@skydoves)
* [AUTOMATIC][Paywalls V2] Updates paywall-preview-resources submodule
(#2613) via RevenueCat Git Bot (@RCGitBot)
* Migrate amazon & debugview modules to KTS (#2327) via Jaewoong Eum
(@skydoves)
* Update to Dokka 2.0.0 (#2609) via Toni Rico (@tonidero)
* Add log when restoring purchases finds no purchases with some
troubleshooting (#2599) via Toni Rico (@tonidero)

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
tonidero pushed a commit that referenced this pull request Aug 25, 2025
- Migrate `amazon` & `debugview` modules to KTS
- Bump kover to 0.7.2 and remove the `afterEvaluate` tasks. (refer to
Kotlin/kotlinx-kover#362)

---------

Co-authored-by: JayShortway <29483617+JayShortway@users.noreply.github.com>
@tonidero tonidero mentioned this pull request Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants