search

Pepita73 / webproghu_dev

Webprog.hu apache-php7.2, Drupal 8.5.5
1 stars 1 forks source link

Coding standards #3

Open Endyl opened 6 years ago

Endyl commented 6 years ago

Amire van drupalos coding standard ott használjuk azt (beleértve css, js, stb.)? Van esetleg más javaslat, igény?

Van más hasonló dolog, amiben meg kéne egyezni, mielőtt elkezdünk össze nem passzoló kódokat gyártani?

Pepita73 commented 6 years ago

Elvben én egyetértek a linkelt standard használatával, de nincs most időm végignyálazni. Nagyjából jónak tűnik, de így 3 perc alatt 2 dolgot már találtam, amiben én el térnék (ha mindenki egyetért):

  1. https://www.drupal.org/docs/develop/standards/coding-standards#functdecl A sorrend ok, de lévén PHP 7.2, kötelezően használni kéne type és return type declaration-t. Rengeteg "invalid argument supplied for foreach" és hasonló hibától képes megkímélni..
  2. https://www.drupal.org/docs/develop/standards/coding-standards#linelength Szerintem ma már nyugodtan lehet 120.
Endyl commented 6 years ago

A type declaration jó ötletnek tűnik. Sorhossznál viszont maradnék a 80-nál, mert úgy a kisebb monitorokon is meg lehet oldani viszonylag kényelmesen a side-by-side paneles szerkesztést :)