Skip to content

Conversation

@lydell
Copy link
Member

@lydell lydell commented Nov 12, 2019

Prettier supports having just a string in the config file to use a
shared config from a package. Example:

"@company/prettier-config"

But the JSON schema generated by running node scripts/generate-schema.js did not include that possibility.

See SchemaStore/schemastore#855 for more
information.

Prettier supports having just a string in the config file to use a
shared config from a package. Example:

    "@company/prettier-config"

But the JSON schema generated by running `node
scripts/generate-schema.js` did not include that possibility.

See SchemaStore/schemastore#855 for more
information.
@alexander-akait
Copy link
Member

Can we add test?

@thorn0
Copy link
Member

thorn0 commented Nov 13, 2019

There already exists a test for this in tests_integration/cli/config/external-config.

@lydell lydell merged commit 44b8191 into master Nov 13, 2019
@lydell lydell deleted the fix-schema-generation branch November 13, 2019 16:46
lipis added a commit that referenced this pull request Jan 8, 2020
* 'master' of github.com:prettier/prettier:
  Don't lowercase element names in CSS selectors (#6947)
  Support string-to-package config in JSON schema: (#6941)
  Rename SECURITY.md to .github/SECURITY.md
  Create SECURITY.md
  Add the XML plugin to the docs (#6944)
  TypeScript: Fix formatting of type operators as arrow function… (#6901)
  SCSS: don't add extra comma after last comment in map (#6918)
  Remove typescript-etw from build (#6919)
@lock lock bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Feb 12, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants