Skip to content
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

feat: blog: Announcing JSON and Markdown linting #642

Merged
merged 6 commits into from
Oct 3, 2024
Merged

Conversation

nzakas
Copy link
Member

@nzakas nzakas commented Oct 1, 2024

Prerequisites checklist

What is the purpose of this pull request?

Added a blog post announcing JSON and Markdown linting plus Code Explorer.

Related Issues

Refs eslint/eslint#16999

Is there anything you'd like reviewers to focus on?

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for new-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/new-eslint/deploys/66fd57758dd51b0008e0ab1a
😎 Deploy Preview https://deploy-preview-642--new-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for es-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/es-eslint/deploys/66fd5775e23a9d00088a170d
😎 Deploy Preview https://deploy-preview-642--es-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for ja-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/ja-eslint/deploys/66fd57753a49300008920073
😎 Deploy Preview https://deploy-preview-642--ja-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for hi-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/hi-eslint/deploys/66fd5775b0631400088adfa7
😎 Deploy Preview https://deploy-preview-642--hi-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for zh-hans-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/zh-hans-eslint/deploys/66fd5775f34a340008e9edd8
😎 Deploy Preview https://deploy-preview-642--zh-hans-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for fr-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/fr-eslint/deploys/66fd5775a3f39000083f601c
😎 Deploy Preview https://deploy-preview-642--fr-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for pt-br-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/pt-br-eslint/deploys/66fd57752b75560008a3ebb5
😎 Deploy Preview https://deploy-preview-642--pt-br-eslint.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 site configuration.

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for de-eslint ready!

Name Link
🔨 Latest commit 79ef54f
🔍 Latest deploy log https://app.netlify.com/sites/de-eslint/deploys/66fd577575dc4a000852a3a1
😎 Deploy Preview https://deploy-preview-642--de-eslint.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 site configuration.

Copy link
Member

@harish-sethuraman harish-sethuraman left a comment

Choose a reason for hiding this comment

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

LGTM ✅

amareshsm
amareshsm previously approved these changes Oct 1, 2024
Copy link
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

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

LGTM. I’ll keep it open for a while in case anyone else has suggestions before we merge.


Code Explorer allows you to view and explore the AST of different languages to help in creating custom rules. For JSON and Markdown, you'll get an expandable view of the AST, while JavaScript allows you to see not just the AST but also the scope and code path information that ESLint generates as it parses your code. Going forward, we'll continue to update Code Explorer with new languages and capabilities designed to help you create custom rules.

And the best part? Code Explorer is [open source](https://github.com/eslint/code-explorer), so you can help us make it even better.
Copy link
Member

Choose a reason for hiding this comment

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

Just a reminder to make the repo public (I'm guessing after this is merged).

Copy link
Member Author

Choose a reason for hiding this comment

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

Yup. Waiting for tomorrow.

Copy link
Member

@mdjermanovic mdjermanovic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Leaving it open until tomorrow.

@nzakas nzakas merged commit 3663e72 into main Oct 3, 2024
37 checks passed
@nzakas nzakas deleted the json-md-code-explorer branch October 3, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

4 participants