aminnairi / esgi-stylelint-config Goto Github PK

Is your feature request related to a problem? Please describe.
The CLI currently does not account for fixable errors.

Describe the solution you'd like
Automatically fix errors where possible.

Describe alternatives you've considered
None.

Additional context
None.

Is your feature request related to a problem? Please describe.
Installing the NPM dependencies for this project should be made for development purposes, not production.

Describe the solution you'd like
Update the document to show how to install the dependencies using devDependencies instead of dependencies.

Describe alternatives you've considered
None.

Additional context
None.

Is your feature request related to a problem? Please describe.
The Node API for StyleLint has a formatter style very basic.

Describe the solution you'd like
Use the CLI formatter in the Node API to have the same feeling of using the StyleLint CLI while using this package's CLI.

Describe alternatives you've considered
None.

Additional context
None.

Is your feature request related to a problem? Please describe.
There is no way to know if this library works for older Node.js versions and it should be stated in the package configuration.

Describe the solution you'd like
A way of testing every major versions of Node to see which one is supported by this package.

Describe alternatives you've considered
None.

Additional context
None.

Is your feature request related to a problem? Please describe.
There is no badge for having a quick look at the current NPM version of this package.

Describe the solution you'd like
Add a badge displaying the current NPM version for this package and a link to go to the NPM website's homepage for this package.

Describe alternatives you've considered
None.

Additional context
None.

Describe the bug
Linting and fixing multiple files does not currently work.

To Reproduce
Steps to reproduce the behavior:

1. Run the command npx @esgi/stylelint-config *.css
2. Display the content of the linted files
3. See that only the first one has been linted & fixed

Expected behavior
All files should be checked and fixed.

Screenshots
None.

Environment (please complete the following information):

• Operating system & version: Arch 5.14.6
• Node.js version: 16.0.0
• Package version 0.3.0

Additional context
This issue is caused by the binary only accounting for the first argument in the process.argv variable instead of slicing everything from the second argument.

Is your feature request related to a problem? Please describe.
There should be a badge displaying the vulnerabilities' count on the documentation.

Describe the solution you'd like
Use Snyk to create a report and display the vulnerabilities' count in the documentation.

Describe alternatives you've considered
Use the NPM command for searching for vulnerabilities (but without any badge or details) in a workflow.

Additional context
None.

Is your feature request related to a problem? Please describe.
There is no way to quickly check CSS files against the ESGI CSS code style without having to setup a whole new NPM project.

Describe the solution you'd like
Using NPX to run the package against one or more css files.

Describe alternatives you've considered
None.

Additional context
None.

Is your feature request related to a problem? Please describe.
There is no clear way to know if the cli supports globs and multiple files arguments.

Describe the solution you'd like
More instructions on how to use the CLI with multiple files.

Describe alternatives you've considered
None.

Additional context
None.

Describe the bug
This package depends on StyleLint for the CLI part and the latter still has potential vulnerabilities that affect this package as well.

To Reproduce
Steps to reproduce the behavior:

1. Go to https://snyk.io/advisor/npm-package/stylelint#security
2. See that the StyleLint package has a vulnerability and that this also affect this package

Expected behavior
The StyleLint dev team should fix this issue and this package should upgrade the version to the fixed one.

Screenshots
None.

Environment (please complete the following information):
N/A.

Additional context
The disclosed vulnerability has no known exploit, so this package still remains safe to use, especially knowing that the vulnerability is not affecting anything that this package is using from the StyleLint package.

Is your feature request related to a problem? Please describe.
There is no way to know if the library is working correctly.

Describe the solution you'd like
Some integration tests that account for when the library is used in a pseudo-production environment using a tarball of the package.

Describe alternatives you've considered
Manual testing.

Additional context
None.

Describe the bug
The rule for function-calc-no-invalid does not exist.

To Reproduce
Steps to reproduce the behavior:
None.

Expected behavior
There should be no unknown rule in the ruleset.

Screenshots
None.

Environment (please complete the following information):

• Operating system & version: Arch Linux 5.14.6
• Node.js version: 16.0.0
• Package version 0.4.1

Additional context
None.

Is your feature request related to a problem? Please describe.
There is no changelog.

Describe the solution you'd like
A list of all changes including the versions and pull requests linked to these.

Describe alternatives you've considered
Use GitHub's changelog page (but being highly coupled to GitHub and not having being able to migrate to Framagit for instance).

Additional context
None.

Is your feature request related to a problem? Please describe.
There is currently no way to know how much this package weights.

Describe the solution you'd like
A badge for using Bundlephobia to display the package size in its mingzipped form.

Describe alternatives you've considered
None.

Additional context
None.