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

[No QA] Swap out uses of string comparisons for fromJSON #5078

Merged
merged 1 commit into from
Sep 6, 2021

Conversation

roryabraham
Copy link
Contributor

cc @Jag96

Details

Consistently implements a new best practice in GitHub Workflows outlined here. Part of post-mortem cleanup from Monday's deploy fire.

Fixed Issues

$ #4929

Tests

  1. Merge this PR
  2. Verify that it does not break any of the altered workflows.

QA Steps

None.

@roryabraham roryabraham self-assigned this Sep 3, 2021
@roryabraham roryabraham requested a review from a team as a code owner September 3, 2021 22:58
@MelvinBot MelvinBot requested review from Gonals and removed request for a team September 3, 2021 22:59
@Gonals Gonals merged commit b3421ce into main Sep 6, 2021
@Gonals Gonals deleted the Rory-RefactorFromJSON branch September 6, 2021 15:04
@OSBotify
Copy link
Contributor

OSBotify commented Sep 6, 2021

✋ 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

OSBotify commented Sep 8, 2021

🚀 Deployed to staging by @Gonals in version: 1.0.94-2 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Sep 9, 2021

🚀 Deployed to production by @yuwenmemon in version: 1.0.95-1 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 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.

3 participants