Skip to content

Fix SDKMAN issues#822

Merged
tonidero merged 3 commits into
mainfrom
fix-sdkman-issues
Mar 6, 2023
Merged

Fix SDKMAN issues#822
tonidero merged 3 commits into
mainfrom
fix-sdkman-issues

Conversation

@tonidero

Copy link
Copy Markdown
Contributor

Description

This PR fixes the issues with SDKMAN outages on CI systems.

@tonidero tonidero added the build label Feb 27, 2023
@codecov

codecov Bot commented Feb 27, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.54%. Comparing base (833994a) to head (8349a65).
Report is 931 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #822   +/-   ##
=======================================
  Coverage   81.54%   81.54%           
=======================================
  Files         121      121           
  Lines        3999     3999           
  Branches      512      512           
=======================================
  Hits         3261     3261           
  Misses        535      535           
  Partials      203      203           

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

@tonidero tonidero added the HOLD label Feb 27, 2023
@tonidero tonidero changed the title [WIP] Fix SDKMAN issues Fix SDKMAN issues Mar 3, 2023
@tonidero tonidero removed the HOLD label Mar 3, 2023
@tonidero tonidero requested a review from a team March 3, 2023 10:30
@tonidero tonidero marked this pull request as ready for review March 3, 2023 10:30
@tonidero

tonidero commented Mar 3, 2023

Copy link
Copy Markdown
Contributor Author

I tested simulating the error with that script and seems like it works fine. I think we can merge this and see how it goes. Tests seem to currently pass fine without installing sdkman.

@tonidero tonidero merged commit e94263c into main Mar 6, 2023
@tonidero tonidero deleted the fix-sdkman-issues branch March 6, 2023 09:12
tonidero pushed a commit that referenced this pull request Mar 9, 2023
**This is an automatic release.**

### Bugfixes
* Fix docs deployment (#836) via Toni Rico (@tonidero)
* Fix issue with missing subscriber attributes if set after login but
before login callback (#809) via Toni Rico (@tonidero)
### Other Changes
* Fix SDKMAN issues (#822) via Toni Rico (@tonidero)

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
tonidero added a commit that referenced this pull request Mar 9, 2023
## 5.8.2

⚠️ ⚠️ ⚠️ ⚠️ 

Android SDK versions 5.8.0 and 5.8.1 have a major bug that prevents
sending purchase tokens to RevenueCat. Users will get charged but won't
receive entitlements. An automatic refund will occur after 72 hours
unless an unaffected SDK version is used. Downgrade to 5.7.1 or upgrade
to 5.8.2 or later to avoid this bug.
Users who got charged while using 5.8.0 and 5.8.1 will be able to access
their purchases automatically upon opening an app that uses 5.7.1 or
lower, or 5.8.2 or higher.

⚠️ ⚠️ ⚠️ ⚠️

### Other changes in 5.8.0 and 5.8.1
### New Features
* Diagnostics (#811) via Toni Rico (@tonidero)
### Bugfixes
* Fix issue with missing subscriber attributes if set after login but
before login callback (#809) via Toni Rico (@tonidero)
### Dependency Updates
* Bump fastlane from 2.212.0 to 2.212.1 (#821) via dependabot[bot]
(@dependabot[bot])
* Bump fastlane from 2.211.0 to 2.212.0 (#808) via dependabot[bot]
(@dependabot[bot])
* Bump fastlane-plugin-versioning_android from 0.1.0 to 0.1.1 (#798) via
dependabot[bot] (@dependabot[bot])
* Bump danger from 8.6.1 to 9.2.0 (#778) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Fix docs deployment (#836) via Toni Rico (@tonidero)
* Fix SDKMAN issues (#822) via Toni Rico (@tonidero)
* Fix bundle install on CircleCI (#827) via Cesar de la Vega (@vegaro)
* Update README.md to include minimum Kotlin version (#786) via Cesar de
la Vega (@vegaro)
* Remove `tag_release_with_latest_if_needed` fastlane lane (#781) via
Cesar de la Vega (@vegaro)
* Adds docs for timeouts when closing and releasing (#759) via Cesar de
la Vega (@vegaro)
* Add Amazon App tester package to purchase tester queries (#789) via
Stefan Wehner (@tonidero)

---------

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
Co-authored-by: Toni Rico <antonio.rico.diez@revenuecat.com>
tonidero added a commit that referenced this pull request Jun 18, 2023
### Description
This PR fixes the issues with SDKMAN outages on CI systems.
@vegaro vegaro added pr:other and removed pr:build labels Sep 17, 2024
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