Skip to content

Revert "Update gems"#774

Merged
joshdholtz merged 2 commits into
mainfrom
revert-765-update-gems
Feb 6, 2023
Merged

Revert "Update gems"#774
joshdholtz merged 2 commits into
mainfrom
revert-765-update-gems

Conversation

@joshdholtz

Copy link
Copy Markdown
Member

Reverts #765

Our CI (especially deploys) break because octokit >= 5.x.x requires Ruby >= 2.7 and we don't have that being used in our jobs

I tried adding them using CircleCi's Ruby Orb but it rvm fails to install newer Ruby versions due to apt-get issues 🤷‍♂️

This reverts commit 3663eb4.
@joshdholtz joshdholtz requested review from tonidero and vegaro February 6, 2023 14:34
@joshdholtz joshdholtz added the pr:fix A bug fix label Feb 6, 2023
Comment thread Gemfile.lock Outdated
GIT
remote: https://github.com/RevenueCat/fastlane-plugin-revenuecat_internal
revision: 738f255ee8c6d4dbbb9ebdda79782ed90529e00a
revision: 92650e40d28b17899ca0cf80c8ba165931e4ab14

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.

This moves back a couple of commits in out plugin, but I think none of those changes are critical 👍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is my latest commit so I think this moves us up 😊

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.

Hmm I think this is the 3rd to last?
image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

You right lol... I'm having a Monday 🤷‍♂️

@codecov

codecov Bot commented Feb 6, 2023

Copy link
Copy Markdown

Codecov Report

Merging #774 (45758b4) into main (3663eb4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #774   +/-   ##
=======================================
  Coverage   81.54%   81.54%           
=======================================
  Files         121      121           
  Lines        3999     3999           
  Branches      512      512           
=======================================
  Hits         3261     3261           
  Misses        535      535           
  Partials      203      203           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@joshdholtz joshdholtz merged commit f091f6b into main Feb 6, 2023
@joshdholtz joshdholtz deleted the revert-765-update-gems branch February 6, 2023 15:05
@joshdholtz joshdholtz mentioned this pull request Feb 6, 2023
2 tasks
joshdholtz added a commit that referenced this pull request Feb 6, 2023
<!-- Thank you for contributing to Purchases! Before pressing the
"Create Pull Request" button, please provide the following: -->

### Checklist
- [x] If applicable, unit tests
- [x] If applicable, create follow-up issues for `purchases-ios` and
hybrids

### Motivation

This will have `octokit` verison `4.x.x` which will make deploys work
again

### Description

See #774
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants