bump metro minimum supported Node.js to 18.18 #146
Triggered via pull request
September 11, 2024 15:56
vzaidman
Status
Cancelled
Total duration
1m 23s
Artifacts
–
build-test-and-deploy.yml
on: pull_request
Matrix: test
Type check, lint, smoke test
41s
Tests with coverage
1m 15s
Deploy
0s
Annotations
3 errors and 2 warnings
|
test (ubuntu-latest, 18.8) / Tests [Node.js 18.8, ubuntu-latest]
Process completed with exit code 1.
|
|
Tests with coverage
Canceling since a higher priority waiting request for 'build-test-and-deploy-refs/pull/1351/merge' exists
|
|
Tests with coverage
The operation was canceled.
|
|
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
|