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

add linters benchmark #31

Conversation

romainmenke
Copy link
Contributor

@romainmenke romainmenke commented Jul 2, 2023

Stylelint uses a very large chunk of the PostCSS API and it's a very large code base.
Adding it to the benchmarks should make it easier to optimize more parts of PostCSS.

At this time there aren't any other linters that I know of, but maybe in the future more can be added. This will always be a bit apples and oranges because a linter does more or less work depending on config and user preferences.


I also updated dependencies and added recent reference results.

@ai ai marked this pull request as ready for review July 2, 2023 11:32
@romainmenke romainmenke force-pushed the add-linters-benchmark--unassuming-liger-ee7f86accd branch from 2d6da9d to d2bc7f5 Compare July 2, 2023 11:32
linters.js Outdated Show resolved Hide resolved
stylecow-plugin-variables: ^5.1.4
stylis: ^4.1.3
through2: ^4.0.2
lockfileVersion: '6.0'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was unintentional.

Is it ok that this changes?
Or should I install a different version of pnpm so that this remains constant?

Copy link
Member

Choose a reason for hiding this comment

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

Don't worry, I will fix it on my own

@ai ai merged commit 3762df9 into postcss:main Jul 6, 2023
@romainmenke romainmenke deleted the add-linters-benchmark--unassuming-liger-ee7f86accd branch July 6, 2023 11:13
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.

2 participants