chriscoderdr / morro-ride-sharing

1 stars 1 forks source link

Fix NX Cloud distributed task execution error for cacheable build tasks #14

Closed chriscoderdr closed 4 weeks ago

chriscoderdr commented 4 weeks ago

                    - Updated nx.json to ensure 'build' tasks for driver-app

and rider-app are cacheable.

Verified the build tasks for driver-app and rider-app are included under cacheableOperations.
Fixed the configuration in project.json to mark 'build' targets as cacheable for both apps.

Resolved the error in NX Cloud that prevented distributed task execution due to non-cacheable tasks.

                This commit closes #12 by ensuring proper cacheable task configuration for distributed builds.

nx-cloud[bot] commented 4 weeks ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3adb03ce5e1556b4bd8fed576b830a9b2cdaea7b. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

✅ Successfully ran 1 target

- [`nx affected -t lint test build`](https://cloud.nx.app/runs/ZgHdsxURuS?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

chriscoderdr commented 4 weeks ago

NoHoist is not working in npm workspaces but I found out in this issue https://github.com/npm/rfcs/issues/287 this was just recently added (maybe?) so I'm gonna use yarn workspace who fully supports it

chriscoderdr commented 4 weeks ago

It's finally working as expected, this took longer than expected due to the mono repo setup NPM Workspace didn't work cause we need to unhoist some packages due to having different versions of the same package across the projects but cannot change it due to being the versions required by these projects