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

[android] ignore the gms version from react-native-http-bridge. #8429

Merged
merged 1 commit into from
Jun 18, 2019

Conversation

mandrigin
Copy link
Contributor

@mandrigin mandrigin commented Jun 18, 2019

react-native-http-bridge has an unbound dependency on GMS,
that sometimes brings unnecessary transitive dependencies to our builds.

This commit forces to ignore this depencency.

status: ready

@mandrigin mandrigin requested a review from a team as a code owner June 18, 2019 10:28
@auto-assign auto-assign bot removed the request for review from a team June 18, 2019 10:29
@status-github-bot
Copy link

status-github-bot bot commented Jun 18, 2019

Pull Request Checklist

  • Have you updated the documentation, if impacted (e.g. docs.status.im)?

@mandrigin
Copy link
Contributor Author

automated QA can also be skipped, if the app builds, it works :-D

@status-im-auto
Copy link
Member

status-im-auto commented Jun 18, 2019

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 495c481 #1 2019-06-18 10:43:16 ~14 min ios 📦 ipa
✔️ 495c481 #1 2019-06-18 10:45:06 ~15 min macos 📦 dmg
✔️ 495c481 #1 2019-06-18 10:54:53 ~25 min android-e2e 📦 apk
✔️ 495c481 #1 2019-06-18 10:55:07 ~25 min android 📦 apk
✔️ 495c481 #1 2019-06-18 10:58:53 ~29 min linux 📦 App
✔️ 495c481 #1 2019-06-18 11:00:34 ~31 min windows 📦 exe
✔️ 23702dc #2 2019-06-18 11:10:04 ~14 min android-e2e 📦 apk

@mandrigin
Copy link
Contributor Author

okay, Android passes, merging

`react-native-http-bridge` has an unbound dependency on GMS,
that sometimes brings unnecessary transitive dependencies to our builds.

This commit forces to ignore this depencency.

Signed-off-by: Igor Mandrigin <i@mandrigin.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants