search

UB-Mannheim / zotkat

Erweiterung von Zotero für die Katalogisierung
GNU Affero General Public License v3.0
45 stars 20 forks source link

Improve Travis #19

Closed zuphilip closed 8 years ago

zuphilip commented 8 years ago

Das beinhaltet den PR #18 von @kba .

zuphilip commented 8 years ago

Leider gibt es noch ein Problem mit dem Parsing des regulären Ausdruckes als JSON String:

npm ERR! Failed to parse json
npm ERR! Unexpected token '*' at 7:64
npm ERR!     "test": "for f in *.js;do sed '1,/^}/ s/.*//' $f|sed 's/\/\*\* BEGIN TEST C
npm ERR!                                                                ^

@kba Wie kann man dies richtig schreiben?

kba commented 8 years ago

Die Asterisks (*) müssten noch mal escapet werden, weil das ja JSON ist. Aber der Befehl wird ohnehin zu lang, ich habe es in ein kleines Shell-Skript ausgelagert und dokumentiert. Siehe https://github.com/kba/zotkat/tree/improve-travis und https://travis-ci.org/kba/zotkat/builds/121977175

zuphilip commented 8 years ago

Okay, vielen Dank. Ich habe einen PR #20 daraus gemacht. Sieht gut aus!