-
-
Notifications
You must be signed in to change notification settings - Fork 990
Fix --fix CLI flag raising unknown value error
#8313
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
Conversation
🦋 Changeset detectedLatest commit: 2ee2280 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
For example, when `--fix foo.css` is given, `foo.css` was previously interpreted as a `--fix` argument instead of a file path. This change tries to consider `foo.css` as an appropriate file path.
--fix CLI flag to raise unknown value error--fix CLI flag raising unknown value error
2aa8976 to
de335c7
Compare
romainmenke
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ybiquitous. LGTM 👍
Nice to see the increased test coverage as well.
jeddy3
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
I can do the release now.
| datasource | package | from | to | | ---------- | --------- | ------- | ------- | | npm | stylelint | 16.13.0 | 16.13.2 | ## [v16.13.2](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16132---2025-01-14) It fixes a regression bug with `--fix`. - Fixed: `--fix` CLI flag raising unknown value error ([#8313](stylelint/stylelint#8313)) ([@ybiquitous](https://github.com/ybiquitous)). ## [v16.13.1](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16131---2025-01-13) It fixes one bug due to an upstream change in our `ignore` dependency. - Fixed: `ignore.default is not a function` ([#8305](stylelint/stylelint#8305)) ([@romainmenke](https://github.com/romainmenke)).
| datasource | package | from | to | | ---------- | --------- | ------- | ------- | | npm | stylelint | 16.13.0 | 16.13.2 | ## [v16.13.2](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16132---2025-01-14) It fixes a regression bug with `--fix`. - Fixed: `--fix` CLI flag raising unknown value error ([#8313](stylelint/stylelint#8313)) ([@ybiquitous](https://github.com/ybiquitous)). ## [v16.13.1](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16131---2025-01-13) It fixes one bug due to an upstream change in our `ignore` dependency. - Fixed: `ignore.default is not a function` ([#8305](stylelint/stylelint#8305)) ([@romainmenke](https://github.com/romainmenke)).
| datasource | package | from | to | | ---------- | --------- | ------- | ------- | | npm | stylelint | 16.13.0 | 16.13.2 | ## [v16.13.2](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16132---2025-01-14) It fixes a regression bug with `--fix`. - Fixed: `--fix` CLI flag raising unknown value error ([#8313](stylelint/stylelint#8313)) ([@ybiquitous](https://github.com/ybiquitous)). ## [v16.13.1](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16131---2025-01-13) It fixes one bug due to an upstream change in our `ignore` dependency. - Fixed: `ignore.default is not a function` ([#8305](stylelint/stylelint#8305)) ([@romainmenke](https://github.com/romainmenke)).
| datasource | package | from | to | | ---------- | --------- | ------- | ------- | | npm | stylelint | 16.13.0 | 16.13.2 | ## [v16.13.2](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16132---2025-01-14) It fixes a regression bug with `--fix`. - Fixed: `--fix` CLI flag raising unknown value error ([#8313](stylelint/stylelint#8313)) ([@ybiquitous](https://github.com/ybiquitous)). ## [v16.13.1](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16131---2025-01-13) It fixes one bug due to an upstream change in our `ignore` dependency. - Fixed: `ignore.default is not a function` ([#8305](stylelint/stylelint#8305)) ([@romainmenke](https://github.com/romainmenke)).
| datasource | package | from | to | | ---------- | --------- | ------- | ------- | | npm | stylelint | 16.13.0 | 16.14.0 | ## [v16.14.0](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16140---2025-01-25) It fixes 8 bugs and adds features to our `Warning` type. - Added: replaced text and corresponding range to `Warning` type ([#8234](stylelint/stylelint#8234)) ([@romainmenke](https://github.com/romainmenke)). - Fixed: `SyntaxError: The requested module 'table'...` ([#8325](stylelint/stylelint#8325)) ([@denisx](https://github.com/denisx)). - Fixed: `RuleContext['fix']` type ([#8309](stylelint/stylelint#8309)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `declaration-property-value-keyword-no-deprecated` autofix of `border-color` and `overflow` ([#8346](stylelint/stylelint#8346)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `declaration-property-value-keyword-no-deprecated` false negatives for `overflow-x` and `overflow-y` ([#8340](stylelint/stylelint#8340)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `declaration-property-value-keyword-no-deprecated` false negatives for `scrollbar-color` ([#8346](stylelint/stylelint#8346)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `declaration-property-value-keyword-no-deprecated` false positives for non-standard syntax values ([#8329](stylelint/stylelint#8329)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `declaration-property-value-keyword-no-deprecated` warning message for `overflow: <keyword> <keyword>` ([#8340](stylelint/stylelint#8340)) ([@Mouvedia](https://github.com/Mouvedia)). - Fixed: `selector-pseudo-class-no-unknown` false positives for `:has-slotted` ([#8332](stylelint/stylelint#8332)) ([@Mouvedia](https://github.com/Mouvedia)). ## [v16.13.2](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16132---2025-01-14) It fixes a regression bug with `--fix`. - Fixed: `--fix` CLI flag raising unknown value error ([#8313](stylelint/stylelint#8313)) ([@ybiquitous](https://github.com/ybiquitous)). ## [v16.13.1](https://github.com/stylelint/stylelint/blob/HEAD/CHANGELOG.md#16131---2025-01-13) It fixes one bug due to an upstream change in our `ignore` dependency. - Fixed: `ignore.default is not a function` ([#8305](stylelint/stylelint#8305)) ([@romainmenke](https://github.com/romainmenke)).
Closes #8301
For example, when
--fix foo.cssis given,foo.csswas previously interpreted as a--fixargument instead of a file path.This change tries to consider
foo.cssas an appropriate file path.