Closed Ahoerstemeier closed 9 years ago
Benestar
commented
10 years ago Hi, besides your great code improvements you also changed some formattings like spacings before and after brackets. Imho we should make sure that all files are formatted the same way. So please consider to create a new pull request only for formatting issues and let the formatting as it is in this pull request.
Ahoerstemeier
commented
10 years ago Hi Bene,
ich habe die Formatierungsänderungen aus dem Pull Request rausoperiert, und dafür ist noch ein weiterer Bugfix mit reingekommen. In meinem Repository gibt es jetzt zusätzlich einen Branch, in dem ich die ganzen Formatierungen, Dokumentation, Parameterchecks etc. gemacht habe, soll ich dazu auch schon einen Pull Request machen, es sind wohl die allermeisten Stellen bearbeitet? Denn ich bin mittlerweile sicher daß es nicht ohne breaking changes geht, daraus eine stabile leicht benutzbare Bibliothek zu machen, im Moment ist die noch in so vielen Dingen so nahe an der Json-API und benutzt kaum typisierte Strings wo enums angebracht wären, reicht interne Verwaltungsstrukturen nach draußen wo redundante Infos drinstehen etc - aber damit würde ich am liebsten erst anfangen wenn beide Repositories wieder auf einem Stand sind, und ich nicht mehrere Branches miteinander synchronisieren muss.
On Mon, Dec 16, 2013 at 9:11 PM, Benestar notifications@github.com wrote:
Hi, besides your great code improvements you also changed some formattings like spacings before and after brackets. Imho we should make sure that all files are formatted the same way. So please consider to create a new pull request only for formatting issues and let the formatting as it is in this pull request.
— Reply to this email directly or view it on GitHubhttps://github.com/Benestar/wikibase.net/pull/1#issuecomment-30695688 .
Andreas Hörstemeier email: andy@hoerstemeier.de www: http://www.hoerstemeier.com blog: http://tambon.blogspot.com
Benestar
commented
10 years ago Sorry für die späte Antwort. Siehe die Kommentare. Ich fände es gut, wenn die Formatierung (z.B. Leerzeichen nach Klammern, Einrückungen, etc.) einheitlich wären. Auf jeden Fall vielen Dank für die Mühe.
Ahoerstemeier
commented
10 years ago Hallo Bene,
die Formatierung ist mit den Default-Einstellungen von der Visual-Studio-Erweiterung CodeMade gemacht - welches auch dafür sorgt daß alles formatiert bleibt, was VS von sich aus nicht macht. Die zusätzlichen Leerzeilen und Umbrüche machen den Code zwar länger, aber auch sehr viel lesbarer. Außerdem kann VS ja auch vieles einklappen. Eigentlich sollten alle Dateien das Schema bekommen haben und damit einheitlich sein, kann aber sein daß noch welche durchgeschlüpft sind. Eventuell sind diese mittlerweile in dem Weiterentwicklungs-Branch dann doch noch geändert worden.
Die Änderung von int auf Int32, bool auf Boolean und string auf String macht technisch keinerlei Unterschied, wohl aber auch wieder bei der Lesbarkeit, denn dann sind diese Typen farblich genauso dargestellt wie alle anderen (hellblau) Typen, und nicht so wie Schlüsselworte (dunkelblau). Mir sind aber sicher auch manche durchgeschlüpft die noch nicht geändert waren - ich hätte es wohl besser durch Suchen&Ersetzen gemacht statt manuell.
Da keinerlei Reaktion mehr von dir kam, habe ich nach noch einen weiteren Branch gemacht auf dem ich nun das Framework weiterentwickle - dort kann ich schon den Quantity-Datentyp benutzen und Qualifiers hinzufügen. Aber auch am Interface habe ich dann schon einiges geändert wie ich es für sinnvoller halte - statt magischer Konstanten benutze ich enum-Typen etc. Es gibt aber immer noch genug Dinge die mir noch nicht gefallen, aber wofür ich noch keine Zeit hatte, denn der eigentliche Bot und seine Daten war noch wichtiger. Aber damit ist der Fork schon deutlich weiter entfernt, inwieweit du das dann noch mergen kannst weiß ich nicht.
Tschues, Andreas
2014-03-27 21:02 GMT+01:00 Benestar notifications@github.com:
Sorry für die späte Antwort. Siehe die Kommentare. Ich fände es gut, wenn die Formatierung (z.B. Leerzeichen nach Klammern, Einrückungen, etc.) einheitlich wären. Auf jeden Fall vielen Dank für die Mühe.
Reply to this email directly or view it on GitHubhttps://github.com/Benestar/wikibase.net/pull/1#issuecomment-38853532 .
Andreas Hörstemeier email: andy@hoerstemeier.de www: http://www.hoerstemeier.com blog: http://tambon.blogspot.com
Mostly Bugfixes