Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Sep 11, 2025

Current Behavior

The inferred docker:build target does not depend on the build target.
Given that most docker build commands rely on the initial application itself to be built first, it makes sense to depend on the build target.

Expected Behavior

docker:build dependsOn build

@Coly010 Coly010 requested a review from a team as a code owner September 11, 2025 12:46
@Coly010 Coly010 self-assigned this Sep 11, 2025
Copy link

vercel bot commented Sep 11, 2025

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

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

Copy link
Contributor

nx-cloud bot commented Sep 11, 2025

View your CI Pipeline Execution ↗ for commit 6f77752

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 24m 37s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 2m 5s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation ✅ Succeeded 40s View ↗

☁️ Nx Cloud last updated this comment at 2025-09-11 13:17:26 UTC

@jaysoo jaysoo merged commit 459dc88 into master Sep 12, 2025
16 checks passed
@jaysoo jaysoo deleted the docker-build-depends-on-build branch September 12, 2025 16:20
Copy link
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 Sep 18, 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