Merge pull request #2278 from feiming/conflict_dist

Change tsc output dir to dist-types

packages/cli/config/eslint.backend.js

@@ -35,7 +35,7 @@ module.exports = {
     ecmaVersion: 2018,
     sourceType: 'module',
   },
-  ignorePatterns: ['.eslintrc.js', '**/dist/**'],
+  ignorePatterns: ['.eslintrc.js', '**/dist/**', '**/dist-types/**'],
   rules: {
     'no-console': 0, // Permitted in console programs
     'new-cap': ['error', { capIsNew: false }], // Because Express constructs things e.g. like 'const r = express.Router()'

packages/cli/config/eslint.js

@@ -39,7 +39,7 @@ module.exports = {
       version: 'detect',
     },
   },
-  ignorePatterns: ['.eslintrc.js', '**/dist/**'],
+  ignorePatterns: ['.eslintrc.js', '**/dist/**', '**/dist-types/**'],
   rules: {
     'import/newline-after-import': 'error',
     'import/no-duplicates': 'warn',

packages/cli/src/commands/clean/clean.ts

@@ -19,5 +19,6 @@ import { paths } from '../../lib/paths';
 
 export default async function clean() {
   await fs.remove(paths.resolveTarget('dist'));
+  await fs.remove(paths.resolveTarget('dist-types'));
   await fs.remove(paths.resolveTarget('coverage'));
 }

packages/cli/src/lib/builder/config.ts

@@ -37,7 +37,7 @@ export const makeConfigs = async (
   options: BuildOptions,
 ): Promise<RollupOptions[]> => {
   const typesInput = paths.resolveTargetRoot(
-    'dist',
+    'dist-types',
     relativePath(paths.targetRoot, paths.targetDir),
     'src/index.d.ts',
   );