[Beta] prePushHook funktioniert nicht wegen Syntax Fehler

[Routing Plus BKG Team]

Hi

wir haben ebenfalls die prePushHook aktiviert. Bisher müssen wir aber immer mit --no-verify arbeiten, wenn wir was in unseren Branch schieben wollen.

Es kommt nämlich folgende Fehlermeldung:

> masterportal@3.0.0-beta1 prePushHook
> node ./node_modules/@masterportal/mpconfigparser/cli.js ./doc/config.json.md ./doc/config.json.de.md && eslint --max-warnings 0 "./**/*.{vue,js}" && npm run test

> masterportal@3.0.0-beta1 test
> mochapack --recursive --webpack-config devtools/webpack.test.js --file devtools/setupUnitTests.js ./src_3_0_0/**/*.spec.js ./addons/addons_3_0_0/*/!(node_modules)/**/*.spec.js

sh: -c: line 0: syntax error near unexpected token `('
sh: -c: line 0: `mochapack --recursive --webpack-config devtools/webpack.test.js --file devtools/setupUnitTests.js ./src_3_0_0/**/*.spec.js ./addons/addons_3_0_0/*/!(node_modules)/**/*.spec.js'
husky - pre-push hook exited with code 1 (error)

‌

Bisher konnten wir aber die Stelle des Syntax Errors nicht ausfindig machen.

Grüße

‌

[Inka Dudek]

Kann es an der Klammer vor node_modules liegen ./addons/addons_3_0_0/*/!(node_modules)/**/*.spec.js?

Wir können das hier nicht testen, da wir kein Linux haben.

[Routing Plus BKG Team]

Hi @Inka Dudek

nachdem wir in der package.json die Klammer um den node_modules entfernt hatten, ging es ohne Probleme.

https://bitbucket.org/geowerkstatt-hamburg/masterportal/src/66071d838aba493a15d37279e2b52f8ce8495173/package.json#lines-22

‌

Stellt das ein Problem unter Windows dar, wenn die Klammern entfernt werden?

Das funktionierende Endresultat wäre also:

"test": "mochapack --recursive --webpack-config devtools/webpack.test.js --file devtools/setupUnitTests.js ./src_3_0_0/**/*.spec.js ./addons/addons_3_0_0/*/!node_modules/**/*.spec.js",

‌

[Inka Dudek]

Unter Windows funktioniert es auch ohne die Klammern! Ich mache uns ein Ticket dazu.

[Inka Dudek]

Ist auf dem branch dev in der Version 2.39.0 gefixt und auf dem branch dev_vue ist es schon gefixt.