Skip to content

Remove tag_release_with_latest_if_needed fastlane lane#781

Merged
vegaro merged 1 commit into
mainfrom
remove-latest-tagging
Feb 27, 2023
Merged

Remove tag_release_with_latest_if_needed fastlane lane#781
vegaro merged 1 commit into
mainfrom
remove-latest-tagging

Conversation

@vegaro

@vegaro vegaro commented Feb 9, 2023

Copy link
Copy Markdown
Member

I believe we are not using the latest tag for anything. We were using it to point to the latest version from the docs

@vegaro vegaro added the ci label Feb 9, 2023
@vegaro vegaro requested a review from a team February 9, 2023 01:22
@vegaro vegaro changed the title Remove tag_release_with_latest_if_needed fastlane lane Remove tag_release_with_latest_if_needed fastlane lane Feb 9, 2023
@codecov

codecov Bot commented Feb 9, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.54%. Comparing base (186189f) to head (d73efc0).
Report is 936 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #781   +/-   ##
=======================================
  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 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm fine with the change, but we should remember to remove the latest tag after this is merged to avoid it becoming outdated.

@aboedo

aboedo commented Feb 9, 2023

Copy link
Copy Markdown
Member

do we actually need it, though? which docs are pointing to it? Maybe we can update the docs instead?
I'm not a big fan of moving a tag around, we've gotten bit by that before.

@vegaro

vegaro commented Feb 9, 2023

Copy link
Copy Markdown
Member Author

I'm not a big fan of moving a tag around

@aboedo we are moving the tag to the latest release on every release already. I don't think the docs ever pointed to it actually. It was intended to be able to link to the latest release from docs.revenuecat.com, but we are not doing that at the moment.

@vegaro

vegaro commented Feb 9, 2023

Copy link
Copy Markdown
Member Author

I actually just checked and it's even pointing to 6.0.0-alpha.3 instead of 5.7.1 (the actual latest), so it's not even working properly 😆

@vegaro vegaro merged commit 816aba5 into main Feb 27, 2023
@vegaro vegaro deleted the remove-latest-tagging branch February 27, 2023 23:09
tonidero added a commit that referenced this pull request Mar 2, 2023
**This is an automatic release.**

### New Features
* Diagnostics (#811) 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 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 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>
@vegaro vegaro added pr:other and removed pr:ci 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