Skip to content

Conversation

@XantreDev
Copy link
Contributor

Pull Request Name

Description

Tries to fix behavior regression

Fixes #338 (comment)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Runed tests
  • Add E2E test that checks lint running more than cssRefreshTime

Test Configuration:

  • OS + version: e.g. macOS Mojave
  • NPM version: ...
  • Node version: ...

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@XantreDev XantreDev force-pushed the fix-classname-regression branch from c36f4c2 to 835cdc8 Compare June 7, 2024 22:00
@francoismassart
Copy link
Owner

@XantreDev,
I've been working on similar issue on this beta release, would you mind trying it in your projects and let me know if everything run smoothly ?

npm i eslint-plugin-tailwindcss@3.17.3-beta.3 -D

@francoismassart
Copy link
Owner

Should be fixed in latest release (3.17.4)

@XantreDev
Copy link
Contributor Author

Sorry. Had other things to do
Btw, I've implement e2e test, which checked error with lint on long run and encountered that my version with fix works slower than original one. Probably because of additional stats call. I would like to commit it and add to the tests

@francoismassart
Copy link
Owner

@XantreDev, no worries, contributions are always welcome.

You can simply create a new PR with the suggested changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants