Skip to content

Commit

Permalink
revert "let prettier choose the correct parser" + let prettirignore w…
Browse files Browse the repository at this point in the history
…ith fodler travis

This reverts commit 7fcd4b0.
  • Loading branch information
JulioJu committed Apr 13, 2018
1 parent 7fcd4b0 commit a12468d
Show file tree
Hide file tree
Showing 6 changed files with 62 additions and 18 deletions.
1 change: 0 additions & 1 deletion generators/client/templates/angular/.prettierignore.ejs
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
node_modules
target
15 changes: 12 additions & 3 deletions generators/client/templates/angular/.prettierrc.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,15 @@ printWidth: 140
singleQuote: true
tabWidth: 4
useTabs: false
# js and ts rules. Prettier is clever, it uses correct rules and correct parser according to file extension.
jsxBracketSameLine: false
arrowParens: avoid
overrides:
- files: "src/**/{*.ts,*.tsx}"
options:
jsxBracketSameLine: false
arrowParens: avoid
parser: typescript
- files: "src/**/*.css"
options:
parser: css
- files: "src/**/*.scss"
options:
parser: scss
1 change: 0 additions & 1 deletion generators/client/templates/react/.prettierignore.ejs
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
node_modules
target
15 changes: 12 additions & 3 deletions generators/client/templates/react/.prettierrc.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,15 @@ printWidth: 140
singleQuote: true
tabWidth: 2
useTabs: false
# js and ts rules. Prettier is clever, it uses correct rules and correct parser according to file extension.
jsxBracketSameLine: false
arrowParens: avoid
overrides:
- files: "src/**/{*.ts,*.tsx}"
options:
jsxBracketSameLine: false
arrowParens: avoid
parser: typescript
- files: "src/**/*.css"
options:
parser: css
- files: "src/**/*.scss"
options:
parser: scss
24 changes: 19 additions & 5 deletions generators/generator-base-private.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,9 @@ const jhipsterUtils = require('./utils');
const constants = require('./generator-constants');
const {
prettierTransform,
prettierOptions
defaultTsPrettierOptions,
defaultCssPrettierOptions,
defaultScssPrettierOptions
} = require('./generator-transforms');

const CLIENT_MAIN_SRC_DIR = constants.CLIENT_MAIN_SRC_DIR;
Expand Down Expand Up @@ -1074,12 +1076,24 @@ module.exports = class extends Generator {
*/
registerClientTransforms(generator = this) {
if (!generator.skipClient) {
const myFilter = filter(['src/**/*.{ts,tsx,scss,css}'], { restore: true });
const typescriptFilter = filter(['src/**/*.{ts,tsx}'], { restore: true });
// this pipe will pass through (restore) anything that doesn't match typescriptFilter
generator.registerTransformStream([
myFilter,
prettierTransform(prettierOptions),
myFilter.restore
typescriptFilter,
prettierTransform(defaultTsPrettierOptions),
typescriptFilter.restore
]);
const scssFilter = filter(['src/**/*.scss'], { restore: true });
generator.registerTransformStream([
scssFilter,
prettierTransform(defaultScssPrettierOptions),
scssFilter.restore
]);
const cssFilter = filter(['src/**/*.css'], { restore: true });
generator.registerTransformStream([
cssFilter,
prettierTransform(defaultCssPrettierOptions),
cssFilter.restore
]);
}
}
Expand Down
24 changes: 19 additions & 5 deletions generators/generator-transforms.js
Original file line number Diff line number Diff line change
@@ -1,14 +1,26 @@
const through = require('through2');
const prettier = require('prettier');

const prettierOptions = {
const defaultPrettierOptions = {
printWidth: 140,
singleQuote: true,
useTabs: false,
// js and ts rules. Prettier is clever, it uses correct rules and correct parser according to file extension.
useTabs: false
};

const defaultTsPrettierOptions = Object.assign(defaultPrettierOptions, {
jsxBracketSameLine: false,
arrowParens: 'avoid',
};
parser: 'typescript',
});

const defaultCssPrettierOptions = Object.assign(defaultPrettierOptions, {
parser: 'css',
});

const defaultScssPrettierOptions = Object.assign(defaultPrettierOptions, {
parser: 'scss',
});


const prettierTransform = function (defaultOptions) {
const transform = (file, encoding, callback) => {
Expand All @@ -30,5 +42,7 @@ const prettierTransform = function (defaultOptions) {

module.exports = {
prettierTransform,
prettierOptions
defaultTsPrettierOptions,
defaultCssPrettierOptions,
defaultScssPrettierOptions
};

0 comments on commit a12468d

Please sign in to comment.