Skip to content

fix(core): restore linkTaskDetails param for backwards compatibility#33874

Merged
FrozenPandaz merged 1 commit intomasterfrom
fix/restore-link-task-details-param
Dec 15, 2025
Merged

fix(core): restore linkTaskDetails param for backwards compatibility#33874
FrozenPandaz merged 1 commit intomasterfrom
fix/restore-link-task-details-param

Conversation

@FrozenPandaz
Copy link
Copy Markdown
Contributor

Current Behavior

The linkTaskDetails parameter was removed from the NxCache constructor in #33843, which broke Nx Cloud since it still passes this parameter.

Expected Behavior

The NxCache constructor should accept the linkTaskDetails parameter (even if unused) to maintain backwards compatibility with Nx Cloud.

Related Issue(s)

Fixes ed09ee1#r172894152

Keep the unused linkTaskDetails parameter in NxCache constructor for
backwards compatibility with Nx Cloud.

Fixes ed09ee1#r172894152
@FrozenPandaz FrozenPandaz requested review from a team as code owners December 15, 2025 20:30
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
nx-dev Ready Ready Preview Dec 15, 2025 8:35pm

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 15, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit bc15d8f
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69406fe26f55f60008b61ca5
😎 Deploy Preview https://deploy-preview-33874--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Dec 15, 2025

View your CI Pipeline Execution ↗ for commit bc15d8f

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 43m 46s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 22s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 11s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-15 21:19:02 UTC

@FrozenPandaz FrozenPandaz enabled auto-merge (squash) December 15, 2025 20:43
@FrozenPandaz FrozenPandaz merged commit 2be4dde into master Dec 15, 2025
20 checks passed
@FrozenPandaz FrozenPandaz deleted the fix/restore-link-task-details-param branch December 15, 2025 21:19
FrozenPandaz added a commit that referenced this pull request Dec 15, 2025
…33874)

## Current Behavior

The `linkTaskDetails` parameter was removed from the `NxCache`
constructor in #33843, which broke Nx Cloud since it still passes this
parameter.

## Expected Behavior

The `NxCache` constructor should accept the `linkTaskDetails` parameter
(even if unused) to maintain backwards compatibility with Nx Cloud.

## Related Issue(s)

Fixes
ed09ee1#r172894152

(cherry picked from commit 2be4dde)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants