Skip to content

fix(docker): handle undefined options when creating graph#33235

Merged
jaysoo merged 1 commit intomasterfrom
docker/undefined-build-target
Oct 24, 2025
Merged

fix(docker): handle undefined options when creating graph#33235
jaysoo merged 1 commit intomasterfrom
docker/undefined-build-target

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Oct 24, 2025

Current Behavior

When @nx/docker is registered via string only (nxJson.plugins: ["@nx/docker"], project graph creation fails because we try to access options.buildTarget.

Expected Behavior

Handle undefined options gracefully, and still create the default target.

Related Issue(s)

Fixes NXC-3320

@Coly010 Coly010 requested a review from a team as a code owner October 24, 2025 12:00
@Coly010 Coly010 requested a review from Cammisuli October 24, 2025 12:00
@Coly010 Coly010 self-assigned this Oct 24, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 24, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Oct 24, 2025 0:11am

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Oct 24, 2025

View your CI Pipeline Execution ↗ for commit a44c454

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

☁️ Nx Cloud last updated this comment at 2025-10-24 12:32:25 UTC

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 24, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit a44c454
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/68fb6a49aa796b0008970af7
😎 Deploy Preview https://deploy-preview-33235--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.

@jaysoo jaysoo merged commit 54bd3c4 into master Oct 24, 2025
19 checks passed
@jaysoo jaysoo deleted the docker/undefined-build-target branch October 24, 2025 13:13
@emiliosheinz
Copy link
Copy Markdown
Contributor

@Coly010 it would be really nice to have this released under a beta tag so that I can unblock myself. Thanks!

FrozenPandaz pushed a commit that referenced this pull request Oct 27, 2025
## Current Behavior
When `@nx/docker` is registered via string only (`nxJson.plugins:
["@nx/docker"]`, project graph creation fails because we try to access
`options.buildTarget`.

## Expected Behavior
Handle undefined `options` gracefully, and still create the default
target.

## Related Issue(s)

Fixes NXC-3320

(cherry picked from commit 54bd3c4)
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 2, 2025

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 Nov 2, 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.

3 participants