Skip to content

chore: update eslint config and others updated#206

Open
Tony133 wants to merge 3 commits intofastify:mainfrom
Tony133:chore/update-eslint-config-and-others-updated
Open

chore: update eslint config and others updated#206
Tony133 wants to merge 3 commits intofastify:mainfrom
Tony133:chore/update-eslint-config-and-others-updated

Conversation

@Tony133
Copy link
Copy Markdown
Member

@Tony133 Tony133 commented Mar 29, 2026

Proposals:

  • Updated codebase to comply with new ESLint + neostandard rules
  • Removed .eslintrc.js, added eslint.config.js with neostandard preset
  • Updated package.json:
    • Replaced standard with eslint + neostandard
    • Updated lint and lint:fix scripts
    • Removed lint:standard script
  • Remove CHANGELOG.md, because it is no longer up to date with the latest updates and changes to the library
  • Updated tsconfig.json:
    • module: commonjs → Node16
    • target: es2015 → es2022
    • moduleResolution: node → Node16
    • Added exclude for node_modules, test, dist
  • Updated README.md:
    • replaced js-standard-style badge with neostandard
  • Disabled ESLint for benchmark files as they contain vendored/legacy third-party code that predates the project's linting standards and is not meant to be refactored. It remains to be seen whether these files should be deleted or updated in the future:
    • bench/dicer/formidable-bench-multipart-parser.js
    • bench/dicer/multipartser-bench-multipart-parser.js
    • bench/dicer/parted-bench-multipart-parser.js
    • bench/dicer/parted-multipart.js

@Tony133 Tony133 marked this pull request as ready for review March 29, 2026 10:49
@Tony133 Tony133 force-pushed the chore/update-eslint-config-and-others-updated branch from cd19248 to 7a05764 Compare March 29, 2026 10:49
@Tony133 Tony133 marked this pull request as draft March 29, 2026 11:13
@Tony133 Tony133 force-pushed the chore/update-eslint-config-and-others-updated branch from 05bf778 to 611f8ab Compare March 29, 2026 14:48
@Tony133 Tony133 marked this pull request as ready for review March 29, 2026 14:55
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.

1 participant