Skip to content

add google maven repo in android project template - #19712

Closed
dulmandakh wants to merge 1 commit into
react:masterfrom
dulmandakh:android-template-google-maven
Closed

add google maven repo in android project template#19712
dulmandakh wants to merge 1 commit into
react:masterfrom
dulmandakh:android-template-google-maven

Conversation

@dulmandakh

Copy link
Copy Markdown
Contributor

This PR will add Google maven repo to RN project template that contains com.android.support:appcompat-v7:26.1.0, thus fixes react-native run-android using version 0.56.0-rc.1

CI is Green - https://circleci.com/gh/dulmandakh/react-native/235

Test Plan

new Android projects will build and run just fine.

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 14, 2018
}
}
dependencies {
classpath 'com.android.tools.build:gradle:2.2.3'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure this won't need to be updated to classpath 'com.android.tools.build:gradle:3.1.3'?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which I guess would require this change too: software-mansion/react-native-svg#584 (comment)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But I'm not entirely sure, so yeah just asking

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm I saw you did a dedicated PR for that #19697

@dulmandakhdulmandakhJun 14, 2018

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch just works for me, you need to add the changes to android/build.gradle file in the root of your project. Please double check

Bumping gradle plugin to 3.x will cause many issues, like requires build-tools 27

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thanks for clarifying, as soon as we have a new RC out I'll test and report back :)

@react-native-botreact-native-bot added ✅Test Plan Missing Changelog This PR appears to be missing a changelog, or they are incorrectly formatted. Platform: Android Android applications. labels Jun 14, 2018
@facebook-github-botfacebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Jun 14, 2018

@facebook-github-botfacebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

hramos pushed a commit that referenced this pull request Jun 14, 2018
Summary:
This PR will add Google maven repo to RN project template that contains com.android.support:appcompat-v7:26.1.0, thus fixes `react-native run-android` using version 0.56.0-rc.1
CI is Green - https://circleci.com/gh/dulmandakh/react-native/235
new Android projects will build and run just fine.
Closes#19712
Differential Revision: D8433730
Pulled By: hramos
fbshipit-source-id: b7d5a1cd5a97b1c4aad2a307158d6dbfcf9a42a5
@dulmandakh
dulmandakh deleted the android-template-google-maven branch June 21, 2018 01:40
grabbou pushed a commit to react-native-community/cli that referenced this pull request Sep 26, 2018
Summary:
This PR will add Google maven repo to RN project template that contains com.android.support:appcompat-v7:26.1.0, thus fixes `react-native run-android` using version 0.56.0-rc.1
CI is Green - https://circleci.com/gh/dulmandakh/react-native/235
new Android projects will build and run just fine.
Closesreact/react-native#19712
Differential Revision: D8433730
Pulled By: hramos
fbshipit-source-id: b7d5a1cd5a97b1c4aad2a307158d6dbfcf9a42a5
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.Import StartedThis pull request has been imported. This does not imply the PR has been approved.Missing ChangelogThis PR appears to be missing a changelog, or they are incorrectly formatted.Platform: AndroidAndroid applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dulmandakh@facebook-github-bot@kelset@react-native-bot