From 9689211df7b2cc190977fab5f84e94342d3c7714 Mon Sep 17 00:00:00 2001
From: Ali Ijaz Sheikh <ofrobots@google.com>
Date: Tue, 5 Jun 2018 01:03:00 +0200
Subject: [PATCH] feat: generate .clang-format (#163)

`init` now generates a .clang-format file. This allows IDEs to pick up
the config and integrate well with the gts provided style.

Fixes: https://github.com/google/ts-style/issues/120
---
 src/format.ts        | 3 +--
 src/init.ts          | 8 +++++++-
 test/test-kitchen.ts | 5 ++---
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/format.ts b/src/format.ts
index 13b2cdfd..ad74203c 100644
--- a/src/format.ts
+++ b/src/format.ts
@@ -18,8 +18,7 @@ import {createProgram} from './lint';
 
 const clangFormat = require('clang-format');
 
-const baseArgs =
-    ['-style', '{Language: JavaScript, BasedOnStyle: Google, ColumnLimit: 80}'];
+const baseArgs = ['-style=file'];
 
 /**
  * Run tslint fix and clang fix with the default configuration
diff --git a/src/init.ts b/src/init.ts
index 1b57c87c..73fac106 100644
--- a/src/init.ts
+++ b/src/init.ts
@@ -156,6 +156,11 @@ async function generateTsConfig(options: Options): Promise<void> {
   return generateConfigFile(options, './tsconfig.json', config);
 }
 
+async function generateClangFormat(options: Options): Promise<void> {
+  const style = await read(path.join(__dirname, '../../.clang-format'), 'utf8');
+  return generateConfigFile(options, './.clang-format', style);
+}
+
 async function generateConfigFile(
     options: Options, filename: string, contents: string) {
   let existing;
@@ -183,7 +188,7 @@ async function generateConfigFile(
     if (!options.dryRun) {
       await write(filename, contents);
     }
-    options.logger.dir(JSON.parse(contents));
+    options.logger.log(contents);
   }
 }
 
@@ -222,6 +227,7 @@ export async function init(options: Options): Promise<boolean> {
   }
   await generateTsConfig(options);
   await generateTsLintConfig(options);
+  await generateClangFormat(options);
 
   // Run `npm install` after initial setup so `npm run check` works right away.
   if (!options.dryRun) {
diff --git a/test/test-kitchen.ts b/test/test-kitchen.ts
index 17b169e7..1b0a378f 100644
--- a/test/test-kitchen.ts
+++ b/test/test-kitchen.ts
@@ -86,11 +86,10 @@ test.serial('init', async t => {
         execOpts);
   }
 
-  // Ensure tsconfig.json got generated.
+  // Ensure config files got generated.
   fs.accessSync(`${stagingPath}/kitchen/tsconfig.json`);
-
-  // Ensure tslint.json got generated.
   fs.accessSync(`${stagingPath}/kitchen/tslint.json`);
+  fs.accessSync(`${stagingPath}/kitchen/.clang-format`);
 
   // Compilation shouldn't have happened. Hence no `build` directory.
   const dirContents = fs.readdirSync(`${stagingPath}/kitchen`);