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

README: Clarify eslint is aspirational #344

Merged

Conversation

johnhawkinson
Copy link
Collaborator

It's been FOUR YEARS since cb38fb5 and the tree is not eslint compliant. Don't gaslight developers into thinking they're doing it wrong when the reality is that style and linting claim is solely aspirational.

I do not appreciate wasting my time trying to figure out what is wrong with my development environment because of stuff like this:

jhawk@lrr recap-chrome % eslint src| tail -4

✖ 16159 problems (16159 errors, 0 warnings)
  14851 errors and 0 warnings potentially fixable with the `--fix` option.

It's been FOUR YEARS since cb38fb5 and the tree is not eslint compliant.
Don't gaslight developers into thinking they're doing it wrong when
the reality is that style and linting claim is solely aspirational.
@johnhawkinson johnhawkinson added the no changelog Override Check Changelog Action. label Sep 27, 2023
@mlissner
Copy link
Member

Setting automerge. Sorry for the confusion and trouble and thanks for fixing it for future developers.

@mlissner mlissner merged commit 034e5d5 into freelawproject:main Sep 27, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Override Check Changelog Action.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants