Skip to content

Conversation

@ybiquitous
Copy link
Member

Which issue, if any, is this issue related to?

Closes #8301

Is there anything in the PR that needs further explanation?

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.

@changeset-bot
Copy link

changeset-bot bot commented Jan 14, 2025

🦋 Changeset detected

Latest commit: 2ee2280

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
stylelint Patch

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.
@ybiquitous ybiquitous changed the title Fix --fix CLI flag to raise unknown value error Fix --fix CLI flag raising unknown value error Jan 14, 2025
@ybiquitous ybiquitous marked this pull request as ready for review January 14, 2025 11:59
Copy link
Member

@romainmenke romainmenke left a 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.

Copy link
Member

@jeddy3 jeddy3 left a 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.

@jeddy3 jeddy3 merged commit c03f407 into main Jan 14, 2025
17 checks passed
@jeddy3 jeddy3 deleted the issue-8301 branch January 14, 2025 12:27
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 17, 2025
| 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)).
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 20, 2025
| 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)).
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 24, 2025
| 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)).
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 25, 2025
| 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)).
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 26, 2025
| 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)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Fix unexpected error Unknown value for flag --fix

5 participants