Skip to content

refactor: rename cpp NativeWorkletsModule to WorkletsModuleProxy#6767

Merged
tjzel merged 2 commits intomainfrom
@tjzel/rename-cpp-native-worklets-module
Nov 27, 2024
Merged

refactor: rename cpp NativeWorkletsModule to WorkletsModuleProxy#6767
tjzel merged 2 commits intomainfrom
@tjzel/rename-cpp-native-worklets-module

Conversation

@tjzel
Copy link
Collaborator

@tjzel tjzel commented Nov 27, 2024

Summary

Analogous to:

NativeWorkletsModule is the name of the ObjC/Java modules generated from WorkletsModule. It's confusing that we name some Cpp object the same way. With this change Cpp NativeWorkletsModule becomes WorkletsModuleProxy, the same name we use to refer to it from JavaScript.

Test plan

CI

@tjzel tjzel added the Check compatibility Trigger a time-consuming compatibility check action label Nov 27, 2024
@tjzel tjzel added this pull request to the merge queue Nov 27, 2024
Merged via the queue into main with commit 0162804 Nov 27, 2024
@tjzel tjzel deleted the @tjzel/rename-cpp-native-worklets-module branch November 27, 2024 16:49
tjzel added a commit that referenced this pull request Dec 13, 2024
## Summary

Analogous to:
- #6766 

`NativeWorkletsModule` is the name of the ObjC/Java modules generated
from `WorkletsModule`. It's confusing that we name some Cpp object the
same way. With this change Cpp `NativeWorkletsModule` becomes
`WorkletsModuleProxy`, the same name we use to refer to it from
JavaScript.

## Test plan

CI
tjzel added a commit that referenced this pull request Dec 13, 2024
## Summary

Analogous to:
- #6766 

`NativeWorkletsModule` is the name of the ObjC/Java modules generated
from `WorkletsModule`. It's confusing that we name some Cpp object the
same way. With this change Cpp `NativeWorkletsModule` becomes
`WorkletsModuleProxy`, the same name we use to refer to it from
JavaScript.

## Test plan

CI
tjzel added a commit that referenced this pull request Dec 13, 2024
## Summary

Analogous to:
- #6766 

`NativeWorkletsModule` is the name of the ObjC/Java modules generated
from `WorkletsModule`. It's confusing that we name some Cpp object the
same way. With this change Cpp `NativeWorkletsModule` becomes
`WorkletsModuleProxy`, the same name we use to refer to it from
JavaScript.

## Test plan

CI
r0h0gg6 pushed a commit to r0h0gg6/react-native-reanimated that referenced this pull request Jul 28, 2025
…tware-mansion#6767)

## Summary

Analogous to:
- software-mansion#6766 

`NativeWorkletsModule` is the name of the ObjC/Java modules generated
from `WorkletsModule`. It's confusing that we name some Cpp object the
same way. With this change Cpp `NativeWorkletsModule` becomes
`WorkletsModuleProxy`, the same name we use to refer to it from
JavaScript.

## Test plan

CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Check compatibility Trigger a time-consuming compatibility check action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants