Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

misc: introduce json/markdown formatter #134

Merged
merged 3 commits into from
May 2, 2019

Conversation

fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Apr 15, 2019

uses Prettier and its VSCode extension to format JSON/JSONC(JSON with comments)/Markdown files.

"tslint.configFile": "tslint.json",
"files.trimTrailingWhitespace": true,
"editor.tabSize": 2,
"editor.insertSpaces": true,
"[typescript]": {
"editor.formatOnSave": true
"editor.formatOnSave": true,
"editor.defaultFormatter": "xaver.clang-format"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to confirm -

  1. It seems like the goal of this change to update vscode settings to change default formatter for ts, cpp, and jsonc files. No other logic change. Correct ?

  2. How do the MD files get formatted automatically too ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. the default formatter for .ts and .cpp are still clang-format. this change only add formatter support for .json and .md
  2. the indent, spaces, empty lines and alignment of table and so on

@hariharans29
Copy link
Member

Please make the change to include the op table generation to commit hook and merge this PR

@fs-eire fs-eire force-pushed the format-json-and-md branch from 4bd7cf1 to 8d7d8e4 Compare April 27, 2019 00:44
@fs-eire fs-eire merged commit 205c54f into microsoft:master May 2, 2019
@fs-eire fs-eire deleted the format-json-and-md branch May 2, 2019 20:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants