Commit 930baeb9 authored by Tim Zallmann's avatar Tim Zallmann Committed by Lukas Eipert

Added possibility to run prettier against one specific directory to make our conversion easier

parent 5f3a219e
...@@ -43,3 +43,17 @@ yarn prettier-all-save ...@@ -43,3 +43,17 @@ yarn prettier-all-save
Formats all files in the repository with Prettier. (This should only be used to test global rule updates otherwise you would end up with huge MR's). Formats all files in the repository with Prettier. (This should only be used to test global rule updates otherwise you would end up with huge MR's).
The source of these Yarn scripts can be found in `/scripts/frontend/prettier.js`. The source of these Yarn scripts can be found in `/scripts/frontend/prettier.js`.
### Scripts during Conversion period
```
node ./scripts/frontend/prettier.js check ./vendor/
```
This will go over all files in a specific folder check it.
```
node ./scripts/frontend/prettier.js save ./vendor/
```
This will go over all files in a specific folder and save it.
...@@ -9,6 +9,8 @@ const getStagedFiles = require('./frontend_script_utils').getStagedFiles; ...@@ -9,6 +9,8 @@ const getStagedFiles = require('./frontend_script_utils').getStagedFiles;
const mode = process.argv[2] || 'check'; const mode = process.argv[2] || 'check';
const shouldSave = mode === 'save' || mode === 'save-all'; const shouldSave = mode === 'save' || mode === 'save-all';
const allFiles = mode === 'check-all' || mode === 'save-all'; const allFiles = mode === 'check-all' || mode === 'save-all';
let dirPath = process.argv[3] || '';
if (dirPath.charAt(dirPath.length-1) !== '/') dirPath += '/';
const config = { const config = {
patterns: ['**/*.js', '**/*.vue', '**/*.scss'], patterns: ['**/*.js', '**/*.vue', '**/*.scss'],
...@@ -39,9 +41,9 @@ prettierIgnore.add( ...@@ -39,9 +41,9 @@ prettierIgnore.add(
const availableExtensions = Object.keys(config.parsers); const availableExtensions = Object.keys(config.parsers);
console.log(`Loading ${allFiles ? 'All' : 'Staged'} Files ...`); console.log(`Loading ${allFiles ? 'All' : 'Selected'} Files ...`);
const stagedFiles = allFiles ? null : getStagedFiles(availableExtensions.map(ext => `*.${ext}`)); const stagedFiles = allFiles || dirPath ? null : getStagedFiles(availableExtensions.map(ext => `*.${ext}`));
if (stagedFiles) { if (stagedFiles) {
if (!stagedFiles.length || (stagedFiles.length === 1 && !stagedFiles[0])) { if (!stagedFiles.length || (stagedFiles.length === 1 && !stagedFiles[0])) {
...@@ -60,6 +62,11 @@ if (allFiles) { ...@@ -60,6 +62,11 @@ if (allFiles) {
const patterns = config.patterns; const patterns = config.patterns;
const globPattern = patterns.length > 1 ? `{${patterns.join(',')}}` : `${patterns.join(',')}`; const globPattern = patterns.length > 1 ? `{${patterns.join(',')}}` : `${patterns.join(',')}`;
files = glob.sync(globPattern, { ignore }).filter(f => allFiles || stagedFiles.includes(f)); files = glob.sync(globPattern, { ignore }).filter(f => allFiles || stagedFiles.includes(f));
} else if (dirPath) {
const ignore = config.ignore;
const patterns = config.patterns.map((item) => {return dirPath + item;});
const globPattern = patterns.length > 1 ? `{${patterns.join(',')}}` : `${patterns.join(',')}`;
files = glob.sync(globPattern, { ignore });
} else { } else {
files = stagedFiles.filter(f => availableExtensions.includes(f.split('.').pop())); files = stagedFiles.filter(f => availableExtensions.includes(f.split('.').pop()));
} }
...@@ -73,7 +80,6 @@ if (!files.length) { ...@@ -73,7 +80,6 @@ if (!files.length) {
console.log(`${shouldSave ? 'Updating' : 'Checking'} ${files.length} file(s)`); console.log(`${shouldSave ? 'Updating' : 'Checking'} ${files.length} file(s)`);
files.forEach(file => { files.forEach(file => {
try { try {
prettier prettier
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment