Skip to content

Use the schema for package.json "prettier" key#600

Merged
CiGit merged 2 commits intoprettier:masterfrom
DJMcNab:package-json-schema
Nov 7, 2018
Merged

Use the schema for package.json "prettier" key#600
CiGit merged 2 commits intoprettier:masterfrom
DJMcNab:package-json-schema

Conversation

@DJMcNab
Copy link
Copy Markdown
Contributor

@DJMcNab DJMcNab commented Oct 13, 2018

This uses the schema located at http://json.schemastore.org/prettierrc for the package.json "prettier" key, so that it can be autocompleted and validated.

The only unresolved question I have is whether the schema file should be in a different folder or have a different name. I think that the new file will be included in the built extension, as the .vscodeignore file does not include it, although I'm not certain.

@svipas
Copy link
Copy Markdown
Contributor

svipas commented Nov 7, 2018

You shouldn't add yourseld to the contributor's list, @prettier decides not you, just saying.

@DJMcNab DJMcNab force-pushed the package-json-schema branch from 1e43260 to 3d673e9 Compare November 7, 2018 06:30
@DJMcNab
Copy link
Copy Markdown
Contributor Author

DJMcNab commented Nov 7, 2018

Fixed!

Thanks and sorry @svipben.

@CiGit
Copy link
Copy Markdown
Member

CiGit commented Nov 7, 2018

Great idea. Thank you!

@CiGit CiGit merged commit 57bb76c into prettier:master Nov 7, 2018
@DJMcNab
Copy link
Copy Markdown
Contributor Author

DJMcNab commented Nov 7, 2018

No problem. One of my biggest pain points with prettier was using the package.json settings, which I think will be better now

anisjonischkeit pushed a commit to anisjonischkeit/prettier-vscode that referenced this pull request Nov 9, 2018
* Use the schema for package.json "prettier" key

* Add changelog entry
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.

3 participants