Skip to content

docs: Updated CSS Support blog post #828

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Nov 4, 2025
Merged

docs: Updated CSS Support blog post #828

merged 2 commits into from
Nov 4, 2025

Conversation

@dwightware4
Copy link
Contributor

@dwightware4 dwightware4 commented Oct 27, 2025

Updated code examples to match current version requirements

Fixes #800

Prerequisites checklist

What is the purpose of this pull request?

Updated code examples provided in this blog post because the old examples are no longer valid and point the reader in the wrong direction.

What changes did you make? (Give an overview)

Changed the examples to match the current examples found in the README file for the relevant configuration.

Related Issues

fixes #800

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

N/A

@dwightware4
docs: Updated CSS Support blog post
5c6008b 
Updated code examples to match current version requirements

Fixes eslint#800
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 27, 2025
@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for zh-hans-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/zh-hans-eslint/deploys/69039c5eda791d0008f2ba3d
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for es-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/es-eslint/deploys/69039c5e4f98170008e58385
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for new-eslint ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/new-eslint/deploys/69039c5ecb62760008e80949
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 27, 2025 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@eslint-github-bot eslint-github-bot bot added the documentation Improvements or additions to documentation label Oct 27, 2025
@eslint-github-bot eslint-github-bot bot mentioned this pull request Oct 27, 2025
Closed
1 task
@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for ja-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/ja-eslint/deploys/69039c5e79775d0008e86039
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for hi-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/hi-eslint/deploys/69039c5eda791d0008f2ba41
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for pt-br-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/pt-br-eslint/deploys/69039c5e7ea58200087887ae
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for fr-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/fr-eslint/deploys/69039c5edcd8a6000890a8af
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@netlify
Copy link

netlify bot commented Oct 27, 2025 •
edited
Loading

✅ Deploy Preview for de-eslint ready!

Name Link
🔨 Latest commit 0372cd0
🔍 Latest deploy log https://app.netlify.com/projects/de-eslint/deploys/69039c5eb52f5b00084eeaa0
😎 Deploy Preview https://deploy-preview-828--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 project configuration.

@Tanujkanti4441 Tanujkanti4441 moved this from Needs Triage to Evaluating in Triage Oct 28, 2025
Tanujkanti4441
Tanujkanti4441 reviewed Oct 28, 2025
View reviewed changes
Copy link
Contributor

@Tanujkanti4441 Tanujkanti4441 left a comment

Choose a reason for hiding this comment

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

Hi @dwightware4, thanks for the PR.

lumirlumir
lumirlumir previously approved these changes Oct 30, 2025
View reviewed changes
Copy link
Member

@lumirlumir lumirlumir 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! I've left one non-blocking suggestion.

Would like @Tanujkanti4441 and @nzakas to verify before merging.

src/content/blog/2025-02-18-eslint-css-support.md Outdated
```

If you're using [Tailwind](https://tailwindcss.com), you can configure most of the custom syntax using the builtin `tailwindSyntax` object, like this:
If you're using [Tailwind](https://tailwindcss.com), you can configure the custom syntax using [tailwind-csstree](https://www.npmjs.com/package/tailwind-csstree), like this:
Copy link
Member

@lumirlumir lumirlumir Oct 30, 2025

Choose a reason for hiding this comment

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

Suggested change
If you're using [Tailwind](https://tailwindcss.com), you can configure the custom syntax using [tailwind-csstree](https://www.npmjs.com/package/tailwind-csstree), like this:
If you're using [Tailwind](https://tailwindcss.com), you can configure the custom syntax using [`tailwind-csstree`](https://www.npmjs.com/package/tailwind-csstree), like this:

Non-blocking suggestion: I think it would be nice if the package name is wrapped in backticks, like in the README.md of @eslint/css :)

Ref: https://github.com/eslint/css?tab=readme-ov-file#configuring-tailwind-syntax

Image

@lumirlumir lumirlumir moved this from Evaluating to Second Review Needed in Triage Oct 30, 2025
Tanujkanti4441
Tanujkanti4441 previously approved these changes Oct 30, 2025
View reviewed changes
Copy link
Contributor

@Tanujkanti4441 Tanujkanti4441 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!

@Tanujkanti4441 Tanujkanti4441 moved this from Second Review Needed to Merge Candidates in Triage Oct 30, 2025
@dwightware4 dwightware4 dismissed stale reviews from Tanujkanti4441 and lumirlumir via 0372cd0 October 30, 2025 17:11
@dwightware4
Copy link
Contributor Author

dwightware4 commented Oct 30, 2025

LGTM, thanks! I've left one non-blocking suggestion.

Would like @Tanujkanti4441 and @nzakas to verify before merging.

Updated with this change.

lumirlumir
lumirlumir approved these changes Oct 31, 2025
View reviewed changes
Copy link
Member

@lumirlumir lumirlumir left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks! (Still, would like @nzakas to verify this before merging.)

nzakas
nzakas approved these changes Nov 4, 2025
View reviewed changes
Copy link
Member

@nzakas nzakas 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!

@nzakas nzakas merged commit f788608 into eslint:main Nov 4, 2025
37 checks passed
@github-project-automation github-project-automation bot moved this from Merge Candidates to Complete in Triage Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nzakas nzakas nzakas approved these changes

@lumirlumir lumirlumir lumirlumir approved these changes

@Tanujkanti4441 Tanujkanti4441 Tanujkanti4441 left review comments

Assignees

No one assigned

Labels

accepted documentation Improvements or additions to documentation

Projects

Triage
Status: Complete

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bug: Blog post with misleading (outdated) instructions

4 participants

@dwightware4 @nzakas @Tanujkanti4441 @lumirlumir