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

Skip to content

fix(core): add missing FileType import for Windows watcher build#34369

Merged
FrozenPandaz merged 1 commit intomasterfrom
fix-windows-build
Feb 6, 2026
Merged

fix(core): add missing FileType import for Windows watcher build#34369
FrozenPandaz merged 1 commit intomasterfrom
fix-windows-build

Conversation

@FrozenPandaz
Copy link
Collaborator

Current Behavior

The Windows build (aarch64-pc-windows-msvc) fails to compile with:

error[E0433]: failed to resolve: use of undeclared type `FileType`
  --> packages\nx\src\native\watch\types.rs:128:55

The FileType type is used inside a #[cfg(target_os = "windows")] block but was not imported.

Expected Behavior

The Windows build compiles successfully. The FileType import is scoped inside the #[cfg(target_os = "windows")] block (matching the existing pattern in the macOS block) so there are no unused imports on any platform.

Related Issue(s)

N/A — build breakage discovered during CI publish workflow.

@FrozenPandaz FrozenPandaz requested review from a team as code owners February 6, 2026 22:56
@netlify
Copy link

netlify bot commented Feb 6, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 43e53da
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/698671842876af00080e6cd0
😎 Deploy Preview https://deploy-preview-34369--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.

@FrozenPandaz FrozenPandaz enabled auto-merge (squash) February 6, 2026 22:56
@nx-cloud
Copy link
Contributor

nx-cloud bot commented Feb 6, 2026

View your CI Pipeline Execution ↗ for commit 43e53da

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ❌ Failed 1h 7m 12s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 48s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 11s 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 2026-02-07 00:08:21 UTC

@netlify
Copy link

netlify bot commented Feb 6, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit 43e53da
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69867184116c340008ce0892
😎 Deploy Preview https://deploy-preview-34369--nx-dev.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.

@FrozenPandaz FrozenPandaz disabled auto-merge February 6, 2026 23:46
@FrozenPandaz FrozenPandaz merged commit 1f5520e into master Feb 6, 2026
21 of 24 checks passed
@FrozenPandaz FrozenPandaz deleted the fix-windows-build branch February 6, 2026 23:47
Copy link
Contributor

@nx-cloud nx-cloud bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nx Cloud has identified a possible root cause for your failed CI:

Our e2e-gradle tests are failing due to an environment issue unrelated to this PR's changes. The foojay toolchain resolver is returning 503 errors, and the identical failures appear in the master branch, confirming this is a pre-existing environmental problem rather than a regression from the Rust FileType import fix.

No code changes were suggested for this issue.

Trigger a rerun:

Rerun CI

Nx Cloud View detailed reasoning on Nx Cloud ↗


🎓 Learn more about Self-Healing CI on nx.dev

@github-actions
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 Feb 12, 2026
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