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: Allow local tasks to extend global tasks. #1085

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Conversation

milesj
Copy link
Collaborator

@milesj milesj commented Sep 29, 2023

Our docs said this was possible but looks like I never finished the implementation.

@github-actions
Copy link

Run report for cf8975e7 (ubuntu-latest, 18)

Total time: 2m 11s | Comparison time: 6m 43s | Estimated savings: 4m 32s (67.5% faster)

Action Time Status Info
🟩 SetupNodeTool(18.18.0) 16.7s Passed
🟩 SyncNodeProject(types) 751.5ms Passed
🟩 InstallNodeDeps(18.18.0) 1m 25s Passed
🟦 RunTarget(types:build) 1.4s Cached
🟩 SyncNodeProject(runtime) 643ms Passed
🟩 SyncNodeProject(website) 903.1ms Passed
🟩 SyncNodeProject(report) 653.3ms Passed
🟦 RunTarget(runtime:build) 814.4ms Cached
🟩 SyncWorkspace 574.9ms Passed
🟦 RunTarget(website:typecheck) 1.6s Cached
🟦 RunTarget(website:test) 1.8s Cached
🟦 RunTarget(website:lint) 1.7s Cached
🟩 RunTarget(website:format) 9.8s Passed
🟦 RunTarget(report:typecheck) 10s Cached
🟦 RunTarget(report:test) 1.9s Cached
🟦 RunTarget(report:lint) 2s Cached
🟩 RunTarget(report:format) 2.5s Passed
🟦 RunTarget(report:build) 2.3s Cached
🟦 RunTarget(runtime:typecheck) 1.8s Cached
🟦 RunTarget(runtime:test) 1.7s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.7s Cached
🟩 RunTarget(runtime:format) 2.3s Passed
🟦 RunTarget(website:build) 5.2s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 18

Variables:

MOON_NODE_VERSION = 18.18.0
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@github-actions
Copy link

Run report for cf8975e7 (ubuntu-latest, 20)

Total time: 2m 10s | Comparison time: 6m 42s | Estimated savings: 4m 31s (67.5% faster)

Action Time Status Info
🟩 SetupNodeTool(20.8.0) 17s Passed
🟩 SyncNodeProject(types) 509.5ms Passed
🟩 InstallNodeDeps(20.8.0) 1m 25s Passed
🟦 RunTarget(types:build) 1.6s Cached
🟩 SyncNodeProject(runtime) 843.4ms Passed
🟩 SyncNodeProject(website) 535.6ms Passed
🟩 SyncNodeProject(report) 541.5ms Passed
🟦 RunTarget(runtime:build) 1.1s Cached
🟩 SyncWorkspace 489.1ms Passed
🟦 RunTarget(website:typecheck) 1.4s Cached
🟦 RunTarget(website:test) 1.9s Cached
🟦 RunTarget(website:lint) 2.1s Cached
🟩 RunTarget(website:format) 9.6s Passed
🟦 RunTarget(report:typecheck) 9.7s Cached
🟦 RunTarget(report:test) 1.9s Cached
🟦 RunTarget(report:lint) 1.6s Cached
🟩 RunTarget(report:format) 2.3s Passed
🟦 RunTarget(report:build) 2.3s Cached
🟦 RunTarget(runtime:typecheck) 1.7s Cached
🟦 RunTarget(runtime:test) 1.8s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.9s Cached
🟩 RunTarget(runtime:format) 2s Passed
🟦 RunTarget(website:build) 4.9s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 20

Variables:

MOON_NODE_VERSION = 20.8.0
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@github-actions
Copy link

Run report for cf8975e7 (ubuntu-latest, 16)

Total time: 2m 2s | Comparison time: 6m 1s | Estimated savings: 3m 58s (65.9% faster)

Action Time Status Info
🟩 SetupNodeTool(16.20.2) 15.5s Passed
🟩 SyncNodeProject(types) 498.7ms Passed
🟩 InstallNodeDeps(16.20.2) 1m 21s Passed
🟦 RunTarget(types:build) 1.2s Cached
🟩 SyncNodeProject(runtime) 635.7ms Passed
🟩 SyncNodeProject(website) 521.6ms Passed
🟩 SyncNodeProject(report) 529.3ms Passed
🟦 RunTarget(runtime:build) 885.3ms Cached
🟩 SyncWorkspace 500.7ms Passed
🟦 RunTarget(website:typecheck) 1.3s Cached
🟦 RunTarget(website:test) 1.4s Cached
🟦 RunTarget(website:lint) 1.5s Cached
🟩 RunTarget(website:format) 9.5s Passed
🟦 RunTarget(report:typecheck) 9.3s Cached
🟦 RunTarget(report:test) 1.4s Cached
🟦 RunTarget(report:lint) 1.4s Cached
🟩 RunTarget(report:format) 2s Passed
🟦 RunTarget(report:build) 2.2s Cached
🟦 RunTarget(runtime:typecheck) 1.4s Cached
🟦 RunTarget(runtime:test) 1.3s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.3s Cached
🟩 RunTarget(runtime:format) 1.7s Passed
🟦 RunTarget(website:build) 5.1s Cached
Environment

OS: Linux
Matrix:

os = ubuntu-latest
node-version = 16

Variables:

MOON_NODE_VERSION = 16.20.2
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@github-actions
Copy link

Run report for cf8975e7 (windows-latest, 20)

Total time: 3m 45s | Comparison time: 8m 10s | Estimated savings: 4m 25s (54.1% faster)

Action Time Status Info
🟩 SetupNodeTool(20.8.0) 21.3s Passed
🟩 SyncNodeProject(types) 324.8ms Passed
🟩 InstallNodeDeps(20.8.0) 2m 55s Passed SLOW
🟦 RunTarget(types:build) 1.2s Cached
🟩 SyncNodeProject(runtime) 382.2ms Passed
🟩 SyncNodeProject(website) 331.1ms Passed
🟩 SyncNodeProject(report) 331.4ms Passed
🟦 RunTarget(runtime:build) 976ms Cached
🟩 SyncWorkspace 323.1ms Passed
🟦 RunTarget(website:typecheck) 1s Cached
🟦 RunTarget(website:test) 1.3s Cached
🟦 RunTarget(website:lint) 1.3s Cached
🟩 RunTarget(website:format) 9.1s Passed
🟦 RunTarget(report:typecheck) 9.1s Cached
🟦 RunTarget(report:test) 1.7s Cached
🟦 RunTarget(report:lint) 2s Cached
🟩 RunTarget(report:format) 2.3s Passed
🟦 RunTarget(report:build) 2.1s Cached
🟦 RunTarget(runtime:typecheck) 1.5s Cached
🟦 RunTarget(runtime:test) 1.3s Cached
And 3 more...
Expanded report
Action Time Status Info
🟦 RunTarget(runtime:lint) 1.4s Cached
🟩 RunTarget(runtime:format) 1.7s Passed
🟦 RunTarget(website:build) 8.2s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 20

Variables:

MOON_NODE_VERSION = 20.8.0
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@github-actions
Copy link

Run report for cf8975e7 (windows-latest, 18)

Total time: 2m 51s | Comparison time: 7m 39s | Estimated savings: 4m 48s (62.8% faster)

Action Time Status Info
🟩 SetupNodeTool(18.18.0) 24.4s Passed
🟩 SyncNodeProject(types) 337.7ms Passed
🟩 InstallNodeDeps(18.18.0) 1m 55s Passed
🟦 RunTarget(types:build) 1.5s Cached
🟩 SyncNodeProject(runtime) 422.6ms Passed
🟩 SyncNodeProject(website) 344.6ms Passed
🟩 SyncNodeProject(report) 344.9ms Passed
🟦 RunTarget(runtime:build) 991.9ms Cached
🟩 SyncWorkspace 353.7ms Passed
🟦 RunTarget(website:typecheck) 1.1s Cached
🟦 RunTarget(website:test) 1.7s Cached
🟦 RunTarget(website:lint) 1.7s Cached
🟩 RunTarget(website:format) 12s Passed
🟦 RunTarget(report:typecheck) 12s Cached
🟦 RunTarget(report:test) 1.6s Cached
🟦 RunTarget(report:lint) 1.5s Cached
🟩 RunTarget(report:format) 2.2s Passed
🟦 RunTarget(report:build) 2.2s Cached
🟦 RunTarget(runtime:typecheck) 1.4s Cached
🟦 RunTarget(runtime:test) 1.4s 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) 7.6s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 18

Variables:

MOON_NODE_VERSION = 18.18.0
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@github-actions
Copy link

Run report for cf8975e7 (windows-latest, 16)

Total time: 2m 20s | Comparison time: 6m 33s | Estimated savings: 4m 12s (64.3% faster)

Action Time Status Info
🟩 SetupNodeTool(16.20.2) 16s Passed
🟩 SyncNodeProject(types) 312.1ms Passed
🟩 InstallNodeDeps(16.20.2) 1m 37s Passed
🟦 RunTarget(types:build) 1.5s Cached
🟩 SyncNodeProject(runtime) 514.8ms Passed
🟩 SyncNodeProject(website) 343.4ms Passed
🟩 SyncNodeProject(report) 343.7ms Passed
🟦 RunTarget(runtime:build) 850.5ms Cached
🟩 SyncWorkspace 311.2ms Passed
🟦 RunTarget(website:typecheck) 1s Cached
🟦 RunTarget(website:test) 1.5s Cached
🟦 RunTarget(website:lint) 1.4s Cached
🟩 RunTarget(website:format) 10.2s Passed
🟦 RunTarget(report:typecheck) 10.2s Cached
🟦 RunTarget(report:test) 1.4s Cached
🟦 RunTarget(report:lint) 1.4s Cached
🟩 RunTarget(report:format) 2s 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.6s Passed
🟦 RunTarget(website:build) 6s Cached
Environment

OS: Windows
Matrix:

os = windows-latest
node-version = 16

Variables:

MOON_NODE_VERSION = 16.20.2
Touched files
nextgen/config/src/inherited_tasks_config.rs
nextgen/config/src/project_config.rs
nextgen/config/tests/project_config_test.rs
nextgen/task-builder/src/lib.rs
nextgen/task-builder/src/tasks_builder.rs
nextgen/task-builder/src/tasks_builder_error.rs
nextgen/task-builder/tests/__fixtures__/builder/extends-unknown/moon.yml
nextgen/task-builder/tests/__fixtures__/builder/extends/moon.yml
nextgen/task-builder/tests/tasks_builder_test.rs
packages/cli/CHANGELOG.md

@milesj milesj merged commit 8253043 into master Sep 29, 2023
25 checks passed
@milesj milesj deleted the fix-extend-global-task branch September 29, 2023 17:46
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