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

Skip to content

Conversation

@Coly010
Copy link
Contributor

@Coly010 Coly010 commented Nov 27, 2025

Current Behavior

The base eslint config will ignore **/dist but not **/out-tsc.
This can cause issues if lint is run after a typecheck which has placed .d.ts files into an out-tsc directory.

Expected Behavior

Base eslint config should ignore **/out-tsc

@Coly010 Coly010 requested review from a team as code owners November 27, 2025 10:59
@Coly010 Coly010 requested a review from jaysoo November 27, 2025 10:59
@Coly010 Coly010 self-assigned this Nov 27, 2025
@vercel
Copy link

vercel bot commented Nov 27, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Nov 27, 2025 1:21pm

@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 02a518b
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69284f5e0e5ad80008edd657
😎 Deploy Preview https://deploy-preview-33639--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
Contributor

nx-cloud bot commented Nov 27, 2025

View your CI Pipeline Execution ↗ for commit 02a518b

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 5m 59s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 31s 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-11-27 13:27:54 UTC

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

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 flaky task in your failed CI:

🔂 Since the failure was identified as flaky, we reran your CI automatically.

Nx Cloud View detailed reasoning in Nx Cloud ↗


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

@Coly010 Coly010 merged commit f87e20f into master Nov 27, 2025
19 checks passed
@Coly010 Coly010 deleted the eslint/ignore-out-tsc branch November 27, 2025 13:50
FrozenPandaz pushed a commit that referenced this pull request Nov 27, 2025
…3639)

## Current Behavior
The base `eslint` config will ignore `**/dist` but not `**/out-tsc`.
This can cause issues if lint is run after a `typecheck` which has
placed `.d.ts` files into an `out-tsc` directory.

## Expected Behavior
Base eslint config should ignore `**/out-tsc`

---------

Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com>
Co-authored-by: Coly010 <[email protected]>
(cherry picked from commit f87e20f)
@github-actions
Copy link
Contributor

github-actions bot commented Dec 3, 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 Dec 3, 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.

4 participants