You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Failing test on master: EnvironmentTest.LoadEnvironmentWithSource: FATAL ERROR: v8::ToLocalChecked Empty MaybeLocal. #40605
@targos Should we have a CI job that tries to build and test with --ninja and/or --node-builtin-modules-path? That would prevent bugs like this in the future, though it might be a lot of wasted CI resources for a low probability bug.
Activity
src: make LoadEnvironment with string work with builtin modules path
test: skip macos sandbox test with builtin modules path
targos commentedon Oct 26, 2021
It happens because this test doesn't work with
--node-builtin-modules-path.Proposed fix: #40607
GeoffreyBooth commentedon Oct 26, 2021
@targos Should we have a CI job that tries to build and test with
--ninjaand/or--node-builtin-modules-path? That would prevent bugs like this in the future, though it might be a lot of wasted CI resources for a low probability bug.targos commentedon Oct 27, 2021
Yeah, I'm not sure it's worth spending those resources for a build path that's not really supported outside of core development.
test: skip macos sandbox test with builtin modules path
src: make LoadEnvironment with string work with builtin modules path
test: skip macos sandbox test with builtin modules path
src: make LoadEnvironment with string work with builtin modules path
test: skip macos sandbox test with builtin modules path
src: make LoadEnvironment with string work with builtin modules path