Skip to content

fix(playground): show ast when lint pass#323

Merged
hardfist merged 1 commit intomainfrom
yj/show-ast
Sep 1, 2025
Merged

fix(playground): show ast when lint pass#323
hardfist merged 1 commit intomainfrom
yj/show-ast

Conversation

@hardfist
Copy link
Copy Markdown
Contributor

@hardfist hardfist commented Sep 1, 2025

No description provided.

Copilot AI review requested due to automatic review settings September 1, 2025 10:23
@netlify
Copy link
Copy Markdown

netlify bot commented Sep 1, 2025

Deploy Preview for rslint ready!

Name Link
🔨 Latest commit 93c3951
🔍 Latest deploy log https://app.netlify.com/projects/rslint/deploys/68b5741eefdb2a0008580b98
😎 Deploy Preview https://deploy-preview-323--rslint.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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes an issue in the playground where the AST (Abstract Syntax Tree) was not being displayed when lint passes succeeded. The fix ensures that source files are properly tracked for encoding even when no lint errors are found.

  • Moved source file tracking logic to execute for all processed files, not just when diagnostics are present
  • Relocated the file tracking code from the diagnostic processing loop to the rule configuration callback

@hardfist hardfist merged commit 852de8f into main Sep 1, 2025
17 checks passed
@hardfist hardfist deleted the yj/show-ast branch September 1, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants