Skip to content

fix(linter): fix generator '@nx/eslint:convert-to-flat-config' on win… #31043

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Teamop
Copy link
Contributor

@Teamop Teamop commented May 5, 2025

…dows

Current Behavior

When running nx g @nx/eslint:convert-to-flat-config on windows, the ignores path is not handled correctly. After converting, the path will have the additional /r

Expected Behavior

When running nx g @nx/eslint:convert-to-flat-config on windows, the ignores path should be correct.

Related Issue(s)

NA

@Teamop Teamop requested a review from a team as a code owner May 5, 2025 14:33
@Teamop Teamop requested a review from meeroslav May 5, 2025 14:33
Copy link

vercel bot commented May 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 5, 2025 2:36pm

Copy link

nx-cloud bot commented May 5, 2025

View your CI Pipeline Execution ↗ for commit fa2c5ad.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 34m 25s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 14s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 53s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-05 15:13:37 UTC

@FrozenPandaz FrozenPandaz added the priority: medium Medium Priority (not high, not low priority) label May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Medium Priority (not high, not low priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants