Skip to content

Commit 7bb92a3

Browse files
cortinicofacebook-github-bot
authored andcommitted
refactor: Remove unnecessary parameter from configureRepositories (facebook#48596)
Summary: Pull Request resolved: facebook#48596 The second parameter of `configureRepositories` was unused. Let's remove it. Changelog: [Internal] [Changed] - refactor: Remove unnecessary parameter from configureRepositories Reviewed By: cipolleschi Differential Revision: D68016105 fbshipit-source-id: 9fa05cd33e2f7a6986cf1fcdef0d75e74f315843
1 parent a98528e commit 7bb92a3

File tree

3 files changed

+15
-15
lines changed

3 files changed

+15
-15
lines changed

packages/gradle-plugin/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/ReactPlugin.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ class ReactPlugin : Plugin<Project> {
6767
val versionString = versionAndGroupStrings.first
6868
val groupString = versionAndGroupStrings.second
6969
configureDependencies(project, versionString, groupString)
70-
configureRepositories(project, reactNativeDir)
70+
configureRepositories(project)
7171
}
7272

7373
configureReactNativeNdk(project, extension)

packages/gradle-plugin/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/utils/DependencyUtils.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ internal object DependencyUtils {
2727
* This method takes care of configuring the repositories{} block for both the app and all the 3rd
2828
* party libraries which are auto-linked.
2929
*/
30-
fun configureRepositories(project: Project, reactNativeDir: File) {
30+
fun configureRepositories(project: Project) {
3131
project.rootProject.allprojects { eachProject ->
3232
with(eachProject) {
3333
if (hasProperty(INTERNAL_REACT_NATIVE_MAVEN_LOCAL_REPO)) {

packages/gradle-plugin/react-native-gradle-plugin/src/test/kotlin/com/facebook/react/utils/DependencyUtilsTest.kt

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class DependencyUtilsTest {
3434
val project = createProject()
3535
project.extensions.extraProperties.set("react.internal.mavenLocalRepo", localMaven.absolutePath)
3636

37-
configureRepositories(project, tempFolder.root)
37+
configureRepositories(project)
3838

3939
assertThat(
4040
project.repositories.firstOrNull {
@@ -48,7 +48,7 @@ class DependencyUtilsTest {
4848
val repositoryURI = URI.create("https://oss.sonatype.org/content/repositories/snapshots/")
4949
val project = createProject()
5050

51-
configureRepositories(project, tempFolder.root)
51+
configureRepositories(project)
5252

5353
assertThat(
5454
project.repositories.firstOrNull {
@@ -62,7 +62,7 @@ class DependencyUtilsTest {
6262
val repositoryURI = URI.create("https://repo.maven.apache.org/maven2/")
6363
val project = createProject()
6464

65-
configureRepositories(project, tempFolder.root)
65+
configureRepositories(project)
6666

6767
assertThat(
6868
project.repositories.firstOrNull {
@@ -76,7 +76,7 @@ class DependencyUtilsTest {
7676
val repositoryURI = URI.create("https://dl.google.com/dl/android/maven2/")
7777
val project = createProject()
7878

79-
configureRepositories(project, tempFolder.root)
79+
configureRepositories(project)
8080

8181
assertThat(
8282
project.repositories.firstOrNull {
@@ -90,7 +90,7 @@ class DependencyUtilsTest {
9090
val repositoryURI = URI.create("https://www.jitpack.io")
9191
val project = createProject()
9292

93-
configureRepositories(project, tempFolder.root)
93+
configureRepositories(project)
9494

9595
assertThat(
9696
project.repositories.firstOrNull {
@@ -105,7 +105,7 @@ class DependencyUtilsTest {
105105
var project = createProject()
106106
project.extensions.extraProperties.set("includeJitpackRepository", "false")
107107

108-
configureRepositories(project, tempFolder.root)
108+
configureRepositories(project)
109109

110110
assertThat(
111111
project.repositories.firstOrNull {
@@ -117,7 +117,7 @@ class DependencyUtilsTest {
117117
project = createProject()
118118
project.extensions.extraProperties.set("react.includeJitpackRepository", "false")
119119

120-
configureRepositories(project, tempFolder.root)
120+
configureRepositories(project)
121121

122122
assertThat(
123123
project.repositories.firstOrNull {
@@ -132,7 +132,7 @@ class DependencyUtilsTest {
132132
var project = createProject()
133133
project.extensions.extraProperties.set("includeJitpackRepository", "true")
134134

135-
configureRepositories(project, tempFolder.root)
135+
configureRepositories(project)
136136

137137
assertThat(
138138
project.repositories.firstOrNull {
@@ -144,7 +144,7 @@ class DependencyUtilsTest {
144144
project = createProject()
145145
project.extensions.extraProperties.set("react.includeJitpackRepository", "true")
146146

147-
configureRepositories(project, tempFolder.root)
147+
configureRepositories(project)
148148

149149
assertThat(
150150
project.repositories.firstOrNull {
@@ -161,7 +161,7 @@ class DependencyUtilsTest {
161161
val project = createProject()
162162
project.extensions.extraProperties.set("react.internal.mavenLocalRepo", localMaven.absolutePath)
163163

164-
configureRepositories(project, tempFolder.root)
164+
configureRepositories(project)
165165

166166
val indexOfLocalRepo =
167167
project.repositories.indexOfFirst {
@@ -180,7 +180,7 @@ class DependencyUtilsTest {
180180
val mavenCentralURI = URI.create("https://repo.maven.apache.org/maven2/")
181181
val project = createProject()
182182

183-
configureRepositories(project, tempFolder.root)
183+
configureRepositories(project)
184184

185185
val indexOfSnapshotRepo =
186186
project.repositories.indexOfFirst {
@@ -200,7 +200,7 @@ class DependencyUtilsTest {
200200
val appProject = ProjectBuilder.builder().withName("app").withParent(rootProject).build()
201201
val libProject = ProjectBuilder.builder().withName("lib").withParent(rootProject).build()
202202

203-
configureRepositories(appProject, tempFolder.root)
203+
configureRepositories(appProject)
204204

205205
assertThat(
206206
appProject.repositories.firstOrNull {
@@ -226,7 +226,7 @@ class DependencyUtilsTest {
226226
repo.content { content -> content.excludeGroup("com.facebook.react") }
227227
}
228228

229-
configureRepositories(appProject, tempFolder.root)
229+
configureRepositories(appProject)
230230

231231
// We need to make sure we have Maven Central defined twice, one by the library,
232232
// and another is the override by RNGP.

0 commit comments

Comments
 (0)