-
Notifications
You must be signed in to change notification settings - Fork 101
support for RN >= 0.56 #7
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
Conversation
Would you have a look into this and merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
require("metro/src/reactNativeTransformer")
is needed.
Also, the documentation should be updated to
module.exports = ({
transformer: {
babelTransformerPath: require.resolve("./transformer")
},
});
Yes I just commited the updated documentation |
@@ -16,8 +16,8 @@ or | |||
|
|||
```diff | |||
module.exports = { | |||
+ getTransformModulePath() { | |||
+ return require.resolve("./transformer") | |||
+ transformer { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
transformer: {
missed a colon
I used this pull request with |
@marsinearth it doesn't work, because you have to compile typescript first. Checkout to this branch, run |
@krszwsk so the upstream transformer is broken here? |
@sytler No, you just have to compile TS first in order to use the package from the repository directly. |
@krszwsk have you tried this transformer on 0.57.7+? (metro@0.48.5) |
@sytler yup, I ran it on 0.57.8 |
@krszwsk which version of javascript-obfuscator are u using? try to run |
Im using this config and somehow it is not working (it works in older project, with RN 0.55.4 (metro 0.30.2)) rn-cli.config.js
transformer.js
error
|
No description provided.