Skip to content

fix: credit package.json prettier string configs#819

Open
V1RE wants to merge 1 commit into
fallow-rs:mainfrom
V1RE:fix/prettier-package-json-string-config
Open

fix: credit package.json prettier string configs#819
V1RE wants to merge 1 commit into
fallow-rs:mainfrom
V1RE:fix/prettier-package-json-string-config

Conversation

@V1RE
Copy link
Copy Markdown

@V1RE V1RE commented May 31, 2026

What

Credit package.json "prettier": "<package>" string configs as referenced dependencies in the Prettier plugin.

Also adds an issue #818 fixture and regression integration test.

Why

Closes #818.

Prettier supports using the package.json "prettier" field as an external config package reference, for example:

{
  "prettier": "@scope/prettier-config"
}

Fallow previously treated that shared config package as an unused dev dependency because the Prettier plugin only handled object-style configs and plugins arrays.

Test plan

  • cargo fmt --all -- --check
  • cargo test --workspace
  • cargo clippy --workspace -- -D warnings

@V1RE V1RE requested a review from BartWaardenburg as a code owner May 31, 2026 17:21
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.

False positive unused dependency for package.json Prettier string config

1 participant