bump metro minimum supported Node.js to 18.18 #147
Triggered via pull request
September 11, 2024 15:58
vzaidman
Status
Failure
Total duration
1m 34s
Artifacts
–
build-test-and-deploy.yml
on: pull_request
Matrix: test
Type check, lint, smoke test
36s
Tests with coverage
1m 19s
Deploy
0s
Annotations
1 error and 2 warnings
|
test (ubuntu-latest, 18.8) / Tests [Node.js 18.8, ubuntu-latest]
Process completed with exit code 1.
|
|
Type check, lint, smoke test:
packages/metro-transform-plugins/src/addParamsToDefineCall.js#L26
Always use `== null` or `!= null` to check for `null` AND `undefined` values (even if you just expect either of them). Within fb we treat them as equal and `== null` checks for both
|
|
Type check, lint, smoke test:
packages/metro-transform-plugins/src/import-export-plugin.js#L142
'withLocation' is already defined
|