Skip to content
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

fix: Fix node global not being available during dependency installs. #1089

Merged
merged 3 commits into from
Oct 1, 2023

Conversation

milesj
Copy link
Collaborator

@milesj milesj commented Oct 1, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (ubuntu-latest, 16)

Total time: 1m 58s | Comparison time: 6m 16s | Estimated savings: 4m 18s (68.5% faster)

Action Time Status Info
🟩 SetupNodeTool(16.20.2) 14.8s Passed
🟩 SyncNodeProject(types) 395.6ms Passed
🟩 InstallNodeDeps(16.20.2) 1m 16s Passed
🟦 RunTarget(types:build) 1.3s Cached
🟩 SyncNodeProject(runtime) 469.7ms Passed
🟩 SyncNodeProject(website) 403.7ms Passed
🟩 SyncNodeProject(report) 411.3ms Passed
🟦 RunTarget(runtime:build) 960ms Cached
🟩 SyncWorkspace 404.1ms Passed
🟦 RunTarget(website:typecheck) 1.3s Cached
🟦 RunTarget(website:test) 1.7s Cached
🟦 RunTarget(website:lint) 1.6s Cached
🟩 RunTarget(website:format) 9.4s Passed
🟦 RunTarget(report:typecheck) 9.4s Cached
🟦 RunTarget(report:test) 1.6s Cached
🟦 RunTarget(report:lint) 1.6s Cached
🟩 RunTarget(report:format) 2s Passed
🟦 RunTarget(report:build) 2.1s Cached
🟦 RunTarget(runtime:typecheck) 1.6s Cached
🟦 RunTarget(runtime:test) 1.6s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.5s Cached
🟩 RunTarget(runtime:format) 1.6s Passed
🟦 RunTarget(website:build) 5.9s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 16

Variables:

MOON_NODE_VERSION = 16.20.2
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (ubuntu-latest, 20)

Total time: 2m 7s | Comparison time: 6m 51s | Estimated savings: 4m 43s (69.1% faster)

Action Time Status Info
🟩 SetupNodeTool(20.8.0) 17s Passed
🟩 SyncNodeProject(types) 568.8ms Passed
🟩 InstallNodeDeps(20.8.0) 1m 19s Passed
🟦 RunTarget(types:build) 1.4s Cached
🟩 SyncNodeProject(runtime) 649.1ms Passed
🟩 SyncNodeProject(website) 564.3ms Passed
🟩 SyncNodeProject(report) 570.1ms Passed
🟦 RunTarget(runtime:build) 883.1ms Cached
🟩 SyncWorkspace 578.1ms Passed
🟦 RunTarget(website:typecheck) 1.5s Cached
🟦 RunTarget(website:test) 1.7s Cached
🟦 RunTarget(website:lint) 1.6s Cached
🟩 RunTarget(website:format) 9.6s Passed
🟦 RunTarget(report:typecheck) 9.6s Cached
🟦 RunTarget(report:test) 1.7s Cached
🟦 RunTarget(report:lint) 1.6s Cached
🟩 RunTarget(report:format) 2s Passed
🟦 RunTarget(report:build) 2.1s Cached
🟦 RunTarget(runtime:typecheck) 1.6s Cached
🟦 RunTarget(runtime:test) 1.6s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.6s Cached
🟩 RunTarget(runtime:format) 1.9s Passed
🟦 RunTarget(website:build) 8.9s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 20

Variables:

MOON_NODE_VERSION = 20.8.0
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (ubuntu-latest, 18)

Total time: 2m 8s | Comparison time: 6m 15s | Estimated savings: 4m 6s (65.7% faster)

Action Time Status Info
🟩 SetupNodeTool(18.18.0) 17.2s Passed
🟩 SyncNodeProject(types) 543.8ms Passed
🟩 InstallNodeDeps(18.18.0) 1m 25s Passed
🟦 RunTarget(types:build) 1.4s Cached
🟩 SyncNodeProject(runtime) 655.5ms Passed
🟩 SyncNodeProject(website) 974.5ms Passed
🟩 SyncNodeProject(report) 697.4ms Passed
🟦 RunTarget(runtime:build) 822ms Cached
🟩 SyncWorkspace 547.2ms Passed
🟦 RunTarget(website:typecheck) 1.3s Cached
🟦 RunTarget(website:test) 1.5s Cached
🟦 RunTarget(website:lint) 1.6s Cached
🟩 RunTarget(website:format) 9.5s Passed
🟦 RunTarget(report:typecheck) 9.5s Cached
🟦 RunTarget(report:test) 1.6s Cached
🟦 RunTarget(report:lint) 1.7s Cached
🟩 RunTarget(report:format) 2.2s Passed
🟦 RunTarget(report:build) 2.1s Cached
🟦 RunTarget(runtime:typecheck) 1.5s Cached
🟦 RunTarget(runtime:test) 1.5s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.4s Cached
🟩 RunTarget(runtime:format) 1.9s Passed
🟦 RunTarget(website:build) 5s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 18

Variables:

MOON_NODE_VERSION = 18.18.0
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (windows-latest, 16)

Total time: 2m 37s | Comparison time: 6m 37s | Estimated savings: 3m 59s (60.3% faster)

Action Time Status Info
🟩 SetupNodeTool(16.20.2) 21s Passed
🟩 SyncNodeProject(types) 676.5ms Passed
🟩 InstallNodeDeps(16.20.2) 1m 50s Passed
🟦 RunTarget(types:build) 1.2s Cached
🟩 SyncNodeProject(runtime) 436.9ms Passed
🟩 SyncNodeProject(website) 391.7ms Passed
🟩 SyncNodeProject(report) 391.9ms Passed
🟦 RunTarget(runtime:build) 687.2ms Cached
🟩 SyncWorkspace 588.7ms Passed
🟦 RunTarget(website:typecheck) 1.1s Cached
🟦 RunTarget(website:test) 1.1s Cached
🟦 RunTarget(website:lint) 1.1s Cached
🟩 RunTarget(website:format) 11.6s Passed
🟦 RunTarget(report:typecheck) 11.6s Cached
🟦 RunTarget(report:test) 1.1s Cached
🟦 RunTarget(report:lint) 1.1s Cached
🟩 RunTarget(report:format) 2s Passed
🟦 RunTarget(report:build) 2.2s Cached
🟦 RunTarget(runtime:typecheck) 1.2s Cached
🟦 RunTarget(runtime:test) 1s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.1s Cached
🟩 RunTarget(runtime:format) 1.7s Passed
🟦 RunTarget(website:build) 5.4s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 16

Variables:

MOON_NODE_VERSION = 16.20.2
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (windows-latest, 18)

Total time: 4m 25s | Comparison time: 9m 15s | Estimated savings: 4m 49s (52.2% faster)

Action Time Status Info
🟩 SetupNodeTool(18.18.0) 31s Passed
🟩 SyncNodeProject(types) 416.9ms Passed
🟩 InstallNodeDeps(18.18.0) 3m 24s Passed SLOW
🟦 RunTarget(types:build) 1.5s Cached
🟩 SyncNodeProject(runtime) 636.8ms Passed
🟩 SyncNodeProject(website) 573.7ms Passed
🟩 SyncNodeProject(report) 573.9ms Passed
🟦 RunTarget(runtime:build) 675.4ms Cached
🟩 SyncWorkspace 409.4ms Passed
🟦 RunTarget(website:typecheck) 920.5ms Cached
🟦 RunTarget(website:test) 1s Cached
🟦 RunTarget(website:lint) 1.1s Cached
🟩 RunTarget(website:format) 11.8s Passed
🟦 RunTarget(report:typecheck) 13.3s Cached
🟦 RunTarget(report:test) 3.2s Cached
🟦 RunTarget(report:lint) 1.9s Cached
🟩 RunTarget(report:format) 2.4s Passed
🟦 RunTarget(report:build) 2.2s Cached
🟦 RunTarget(runtime:typecheck) 1.6s Cached
🟦 RunTarget(runtime:test) 1.7s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.3s Cached
🟩 RunTarget(runtime:format) 1.8s Passed
🟦 RunTarget(website:build) 5.7s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 18

Variables:

MOON_NODE_VERSION = 18.18.0
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

Run report for 2e98e9ec (windows-latest, 20)

Total time: 2m 21s | Comparison time: 6m 22s | Estimated savings: 4m (62.9% faster)

Action Time Status Info
🟩 SetupNodeTool(20.8.0) 21.2s Passed
🟩 SyncNodeProject(types) 429.5ms Passed
🟩 InstallNodeDeps(20.8.0) 1m 35s Passed
🟦 RunTarget(types:build) 1.5s Cached
🟩 SyncNodeProject(runtime) 579.4ms Passed
🟩 SyncNodeProject(website) 474.2ms Passed
🟩 SyncNodeProject(report) 474.3ms Passed
🟦 RunTarget(runtime:build) 868.1ms Cached
🟩 SyncWorkspace 428.3ms Passed
🟦 RunTarget(website:typecheck) 1.1s Cached
🟦 RunTarget(website:test) 1.4s Cached
🟦 RunTarget(website:lint) 1.4s Cached
🟩 RunTarget(website:format) 9.2s Passed
🟦 RunTarget(report:typecheck) 9.2s Cached
🟦 RunTarget(report:test) 1.5s Cached
🟦 RunTarget(report:lint) 1.5s Cached
🟩 RunTarget(report:format) 1.9s Passed
🟦 RunTarget(report:build) 2.1s Cached
🟦 RunTarget(runtime:typecheck) 1.5s Cached
🟦 RunTarget(runtime:test) 1.4s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.3s Cached
🟩 RunTarget(runtime:format) 1.8s Passed
🟦 RunTarget(website:build) 5.2s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 20

Variables:

MOON_NODE_VERSION = 20.8.0
Touched files
.yarn/versions/fdfb2523.yml
crates/cli/tests/run_node_test.rs
crates/core/test-utils/src/configs.rs
crates/core/tool/src/lib.rs
crates/node/platform/src/actions/run_target.rs
crates/node/tool/src/node_tool.rs
crates/node/tool/src/npm_tool.rs
crates/node/tool/src/pnpm_tool.rs
crates/node/tool/src/yarn_tool.rs
crates/rust/tool/src/rust_tool.rs
packages/cli/CHANGELOG.md
tests/fixtures/node/base/execBin.js
tests/fixtures/node/base/moon.yml
tests/fixtures/node/postinstall/package.json
tests/fixtures/node/postinstall/postinstall.js

@milesj milesj merged commit c318505 into master Oct 1, 2023
25 checks passed
@milesj milesj deleted the fix-1088 branch October 1, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant