-
Notifications
You must be signed in to change notification settings - Fork 3k
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] Add jcenter back to build.gradle #7191
Conversation
Requesting review from @Jag96 to unblock NewDot Android deploys. |
Discussed in slack, link to PR where this was originally removed: #6914 (comment). |
|
Since we don't have any Android E2E tests I would say no need to wait for the iOS E2E tests. We know that this gradle config is not going to affect iOS. |
Going to try this out and see if it works |
Skipping unnecessary E2E |
@Jag96 looks like this was merged without passing tests. Please add a note explaining why this was done and remove the |
[No QA] Add jcenter back to build.gradle (cherry picked from commit c1ceb3b)
Build running here |
🚀 Cherry-picked to staging by @Jag96 in version: 1.1.29-4 🚀
|
Details
This temporary fix should hopefully unblock Android deploys. More context in slack
Fixed Issues
$ n/a
Tests
Verify that the Android dev build runs. I've never succeeded in locally creating an Android production build, so I think it'll be easier to just merge this and see if it works.
QA Steps
None.
Tested On
Screenshots
Android