Skip to content
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

CLD-7165 Bump MacOS runner version #7828

Merged
merged 12 commits into from Feb 22, 2024
Merged

CLD-7165 Bump MacOS runner version #7828

merged 12 commits into from Feb 22, 2024

Conversation

mvitale1989
Copy link
Member

@mvitale1989 mvitale1989 commented Feb 13, 2024

Summary

This PR bumps the MacOS runner version, in order to use a more recent iOS SDK.

The pipelines will also now utilize the .ruby-version file, instead of specifying an explicit Ruby version to use themselves, to make future Ruby upgrades easier.

Ticket Link

https://mattermost.atlassian.net/browse/CLD-7165

Device Information

This PR was tested on: TBD

Release Note

Bump iOS SDK to 17.2

@mvitale1989 mvitale1989 added the Build App for iOS Build the mobile app for iOS label Feb 13, 2024
@mvitale1989 mvitale1989 added Build App for iOS Build the mobile app for iOS and removed Build App for iOS Build the mobile app for iOS labels Feb 13, 2024
@mvitale1989 mvitale1989 added Build App for iOS Build the mobile app for iOS and removed Build App for iOS Build the mobile app for iOS labels Feb 13, 2024
@yasserfaraazkhan yasserfaraazkhan added Build App for iOS Build the mobile app for iOS and removed Build App for iOS Build the mobile app for iOS labels Feb 13, 2024
@mvitale1989 mvitale1989 added Build App for iOS Build the mobile app for iOS and removed Build App for iOS Build the mobile app for iOS labels Feb 16, 2024
@mvitale1989 mvitale1989 added Build App for iOS Build the mobile app for iOS and removed Build App for iOS Build the mobile app for iOS labels Feb 16, 2024
@mvitale1989 mvitale1989 added Build App for iOS Build the mobile app for iOS Build App for Android Build the mobile app for Android Build Apps for PR Build the mobile app for iOS and Android to test and removed Build App for iOS Build the mobile app for iOS Build App for Android Build the mobile app for Android labels Feb 16, 2024
ios/Gemfile Outdated Show resolved Hide resolved
ios/Gemfile.lock Outdated Show resolved Hide resolved
ios/Podfile.lock Outdated Show resolved Hide resolved
@mvitale1989 mvitale1989 added Build Apps for PR Build the mobile app for iOS and Android to test and removed Build Apps for PR Build the mobile app for iOS and Android to test labels Feb 21, 2024
@mvitale1989 mvitale1989 added Build Apps for PR Build the mobile app for iOS and Android to test and removed Build Apps for PR Build the mobile app for iOS and Android to test labels Feb 21, 2024
@mvitale1989 mvitale1989 marked this pull request as ready for review February 21, 2024 18:35
ios/Gemfile.lock Outdated Show resolved Hide resolved
fastlane/Gemfile.lock Outdated Show resolved Hide resolved
@mvitale1989 mvitale1989 added Build Apps for PR Build the mobile app for iOS and Android to test and removed Build Apps for PR Build the mobile app for iOS and Android to test labels Feb 22, 2024
@enahum enahum added the 4: Reviews Complete All reviewers have approved the pull request label Feb 22, 2024
@mvitale1989 mvitale1989 merged commit c0b2c95 into main Feb 22, 2024
10 checks passed
@mvitale1989 mvitale1989 deleted the cld-7165-bump-ios-sdk branch February 22, 2024 10:35
@amyblais amyblais added this to the v2.15.0 milestone Feb 22, 2024
@amyblais amyblais added the Docs/Needed Requires documentation label Mar 11, 2024
@cwarnermm
Copy link
Member

@mvitale1989 - Are there developer or product documentation updates needed for this PR from your perspective?

@mvitale1989
Copy link
Member Author

I went through the readme, the developer setup docs and the build docs and I don't see any mention of SDK versioning. This change was required to be able to push to the App store, and does not impact app building. In fact users are even free to use older SDKs and they would still work afaik.

So I would say we don't really need to document this. cc @enahum please let me know if you have different thoughts on this

@enahum
Copy link
Contributor

enahum commented Mar 21, 2024

No need to document this :)

@amyblais amyblais added Docs/Not Needed Does not require documentation and removed Docs/Needed Requires documentation labels Apr 1, 2024
cyrusjc pushed a commit to cyrusjc/mattermost-mobile that referenced this pull request May 18, 2024
* CLD-7165 Bump MacOS runner version
* Bump Ruby version
* Adapt pipelines to utilize the ruby version in .ruby-version

---------

Co-authored-by: Daniel Espino <larkox@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Build Apps for PR Build the mobile app for iOS and Android to test Docs/Not Needed Does not require documentation release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants