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

Deploy iOS to production when deploying other platforms #4567

Merged
merged 2 commits into from
Aug 12, 2021
Merged

Conversation

AndrewGable
Copy link
Contributor

Details

For some reason we cannot remember, we were not running iOS deploys when we were deploying every other platform to production. This code change will allow iOS to be deployed to production automatically.

Tests

  1. Merge this PR
  2. Verify the correct iOS version is put into production review when we close a staging QA check list

@AndrewGable AndrewGable self-assigned this Aug 11, 2021
@AndrewGable AndrewGable requested a review from a team as a code owner August 11, 2021 17:35
@MelvinBot MelvinBot requested review from MariaHCD and removed request for a team August 11, 2021 17:36
@MariaHCD MariaHCD merged commit b93e158 into main Aug 12, 2021
@MariaHCD MariaHCD deleted the andrew-prod-ios branch August 12, 2021 06:33
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @MariaHCD in version: 1.0.85-10 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@isagoico
Copy link

@AndrewGable can this be checked off the list?

@AndrewGable
Copy link
Contributor Author

Yes - You can check this one off, thank you!

Noting here that this is the run: https://github.com/Expensify/App/runs/3353546458?check_suite_focus=true

Doesn't look like it worked as expected, but I will create a follow up issue

@AndrewGable
Copy link
Contributor Author

Actually, looks like the code wasn't updated yet.. So we have to wait to test this:

https://github.com/Expensify/App/actions/runs/1140313603/workflow#L211-L214

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants