Skip to content

Commit 6b67920

Browse files
committed
refactor: cleanup module templates
1 parent 109bd9b commit 6b67920

File tree

4 files changed

+9
-37
lines changed

4 files changed

+9
-37
lines changed

packages/create-react-native-library/templates/kotlin-library-new/android/src/main/java/com/{%- project.package_dir %}/{%- project.name %}Module.kt

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,12 @@ package com.<%- project.package %>
33
import com.facebook.react.bridge.ReactApplicationContext
44
import com.facebook.react.module.annotations.ReactModule
55

6-
@ReactModule(name = <%- project.name -%>Module.NAME)
76
class <%- project.name -%>Module(reactContext: ReactApplicationContext) :
87
Native<%- project.name -%>Spec(reactContext) {
98

10-
override fun getName(): String {
11-
return NAME
12-
}
13-
149
// Example method
1510
// See https://reactnative.dev/docs/native-modules-android
1611
override fun multiply(a: Double, b: Double): Double {
1712
return a * b
1813
}
19-
20-
companion object {
21-
const val NAME = "<%- project.name -%>"
22-
}
2314
}
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,17 @@
11
package com.<%- project.package %>
22

3-
import com.facebook.react.ReactPackage
3+
import com.facebook.react.BaseReactPackage
44
import com.facebook.react.bridge.NativeModule
55
import com.facebook.react.bridge.ReactApplicationContext
6+
import com.facebook.react.module.model.ReactModuleInfoProvider
67
import com.facebook.react.uimanager.ViewManager
7-
import java.util.ArrayList
88

9-
class <%- project.name -%>ViewPackage : ReactPackage {
9+
class <%- project.name -%>ViewPackage : BaseReactPackage() {
1010
override fun createViewManagers(reactContext: ReactApplicationContext): List<ViewManager<*, *>> {
11-
val viewManagers: MutableList<ViewManager<*, *>> = ArrayList()
12-
viewManagers.add(<%- project.name -%>ViewManager())
13-
return viewManagers
11+
return listOf(<%- project.name -%>ViewManager())
1412
}
1513

16-
override fun createNativeModules(reactContext: ReactApplicationContext): List<NativeModule> {
17-
return emptyList()
18-
}
14+
override fun getModule(name: String, reactContext: ReactApplicationContext): NativeModule? = null
15+
16+
override fun getReactModuleInfoProvider(): ReactModuleInfoProvider = ReactModuleInfoProvider { emptyMap() }
1917
}

packages/create-react-native-library/templates/native-common/android/build.gradle

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -111,35 +111,18 @@ android {
111111
}
112112
}
113113

114-
lintOptions {
114+
lint {
115115
disable "GradleCompatible"
116116
}
117117

118118
compileOptions {
119119
sourceCompatibility JavaVersion.VERSION_1_8
120120
targetCompatibility JavaVersion.VERSION_1_8
121121
}
122-
123-
sourceSets {
124-
main {
125-
java.srcDirs += [
126-
"generated/java",
127-
"generated/jni"
128-
]
129-
}
130-
}
131122
}
132123

133-
repositories {
134-
mavenCentral()
135-
google()
136-
}
137-
138-
def kotlin_version = getExtOrDefault("kotlinVersion")
139-
140124
dependencies {
141125
implementation "com.facebook.react:react-android"
142-
implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
143126
<% if (project.moduleConfig === 'nitro-modules' || project.viewConfig === 'nitro-view') { -%>
144127
implementation project(":react-native-nitro-modules")
145128
<% } -%>

packages/create-react-native-library/templates/objc-view-new/ios/{%- project.name %}View.mm

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ - (instancetype)initWithFrame:(CGRect)frame
3535

3636
- (void)updateProps:(Props::Shared const &)props oldProps:(Props::Shared const &)oldProps
3737
{
38-
const auto &oldViewProps = *std::static_pointer_cast<<%- project.name -%>ViewProps const>(_props);
38+
const auto &oldViewProps = *std::static_pointer_cast<<%- project.name -%>ViewProps const>(oldProps);
3939
const auto &newViewProps = *std::static_pointer_cast<<%- project.name -%>ViewProps const>(props);
4040

4141
if (oldViewProps.color != newViewProps.color) {

0 commit comments

Comments
 (0)