Skip to content

Commit

Permalink
Optional postinstall script should be cross-platform
Browse files Browse the repository at this point in the history
There is no cross-platform way to check if the file/directory exists that doesn't depend on adding another NodeJS dependency.
This "skip if failed" approach is more noisy, but it should work everywhere.
  • Loading branch information
ybnd committed Apr 24, 2024
1 parent 80be4fc commit 728b561
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
"env:yaml": "ts-node --project ./tsconfig.ts-node.json scripts/env-to-yaml.ts",
"base-href": "ts-node --project ./tsconfig.ts-node.json scripts/base-href.ts",
"check-circ-deps": "npx madge --exclude '(bitstream|bundle|collection|config-submission-form|eperson|item|version)\\.model\\.ts$' --circular --extensions ts ./",
"postinstall": "if [ -f lint/tsconfig.json ]; then yarn build:lint ; fi"
"postinstall": "yarn build:lint || echo 'Skipped DSpace ESLint plugins.'"
},
"browser": {
"fs": false,
Expand Down

0 comments on commit 728b561

Please sign in to comment.