Skip to content

Commit 16ea78f

Browse files
committed
refactor: move vscode settings setup to pnpm postinstall hook
- Remove vscodesettings.js execution from make install targets - Add .vscode/settings.json to .gitignore - Remove .vscode/settings.json from git tracking - Add postinstall hook to package.json to run vscodesettings.js - Fix script to properly check if settings.json exists before reading
1 parent ad06e85 commit 16ea78f

File tree

6 files changed

+8
-11
lines changed

6 files changed

+8
-11
lines changed

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,9 @@ lib64
4444
# locales
4545
locales/*.json
4646

47+
# VSCode
48+
.vscode/settings.json
49+
4750
# Tests
4851
/tests/bin
4952
/tests/develop-eggs

.vscode/settings.json

Lines changed: 0 additions & 5 deletions
This file was deleted.

Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,7 @@ clean: ## Clean development environment
7575

7676
.PHONY: install
7777
install: ## Set up development environment
78-
# Setup ESlint for VSCode
7978
pnpm i
80-
node packages/scripts/vscodesettings.js
8179
make build-deps
8280

8381
##### Documentation

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
"private": true,
44
"scripts": {
55
"preinstall": "npx only-allow pnpm",
6+
"postinstall": "node packages/scripts/vscodesettings.js",
67
"watch": "pnpm --filter @plone/registry --filter @plone/client --filter @plone/components --filter @plone/providers watch",
78
"build:deps": "pnpm --filter @plone/registry build",
89
"build:all": "pnpm --filter @plone/registry --filter @plone/client --filter @plone/components --filter @plone/providers --filter @plone/react-router build",

packages/scripts/vscodesettings.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,12 @@ import fs from 'fs';
33
import { parse, stringify } from 'comment-json';
44

55
let vscodeSettingsJSON;
6-
if (fs.existsSync('.vscode')) {
6+
if (fs.existsSync('.vscode') && fs.existsSync('.vscode/settings.json')) {
77
vscodeSettingsJSON = parse(fs.readFileSync('.vscode/settings.json', 'utf8'));
88
} else {
9-
fs.mkdirSync('.vscode');
9+
if (!fs.existsSync('.vscode')) {
10+
fs.mkdirSync('.vscode');
11+
}
1012
fs.writeFileSync('.vscode/settings.json', '{}');
1113
vscodeSettingsJSON = parse(fs.readFileSync('.vscode/settings.json', 'utf8'));
1214
}

packages/volto/Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,6 @@ clean: ## Clean development environment
6161

6262
.PHONY: install
6363
install: ## Set up development environment
64-
# Setup ESlint for VSCode
65-
node packages/scripts/vscodesettings.js
6664
pnpm i
6765
make build-deps
6866

0 commit comments

Comments
 (0)