search

ffnord / ffnord-puppet-gateway

Deploy and manage your Freifunk community gateway, mostly compatible with Gluon.
15 stars 13 forks source link

Anpassungen zum folgen des Style Guide von Puppet #176

Closed MTRNord closed 7 years ago

MTRNord commented 7 years ago

Ich habe mich mal dran gesetzt und alle lint fehler bezüglich des style guide gefixt (keine klassen namen wurden verändert).

Diese Pull Request ist aktuell noch Work in Progress und ist rein optional. Zudem hab ich es nicht getestet ob dadurch Fehler passieren. Grundsätzlich, da ich nur style geändert habe und keine namen etc., sollte alles wie vorher funktionieren.

Der genutzte Style-Guide ist hier zu finden: http://puppet-lint.com/

MTRNord commented 7 years ago

Die Fehler die nun noch bei travis angezeigt werden sind nun entweder noch style fehler, die parameter oder class namen verändern würden oder die firewall.pp wo ich nicht weiß wie das korrekt wäre aufgrund der mehrzeiligen variablen.

MTRNord commented 7 years ago

Hab gerade die commits kaputt gemacht :) Do Not Push! xD FIXED

sargon commented 7 years ago

Look good for me, so far. Do you want to put more effort into this, or shall I merge it?

MTRNord commented 7 years ago

@sargon as i am not going to work on firewall.pp (because i don't understand it xD) it should be ready to merge

MTRNord commented 7 years ago

Wobei sich da zeilen angaben auch ändern. bin nicht sicher ob deswegen andere pulls kaputt gehen