Closed Kubo2 closed 8 years ago
Waited six days for a review and discussion. Now as a small non-breaking change will be merged. PR čakala šesť dní na kontrolu a diskusiu. Teraz bude ako malá a nerozbíjajúca zmena začlenená.
Reviewed 4 of 4 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.
Comments from the review on Reviewable.io
Z ďalších by som tu videl ešte
web/whatandhow.php
web/about-game.php
Čo sú svojím spôsobom čiastkové duplikáty toho druhého (ale treba sa na nich asi hlbšie pozrieť).
Ďalšie:
web/access-log.php
web/domain-redirect.php
Neskôr by mohla ísť preč aj šablóna web/includes/submenu.php
, ale teraz to nie je dôležité, pretože na tých dvoch BOMoch (U+FEFF
) na začiatku príslušného súboru závisí v podstate omylom layout celého webu.
Viď #7
web/-rules.html
sú nikdy nedokončené pravidlá Diggy's Helper fóraweb/.config.ini
, pôvodne zamýšľaný pre vytvorenie konfigurovateľného DH, nebol nikdy reálne použitýweb/__sub-vyvoj__.php
bol v podstate polyfil neschopnosti WEDOSu poskytnúť viacero subdomén na k jednému hostingu (našťastie už sme prešli k WebSupportu)web/uploadcatalog.php
je súbor, ktorý nemá absolútne nič spoločné s Diggy's Helperom a dostal sa tam práve v čase, keď som nerozvážne pred tromi rokmi commitol novšiu verziu projektu, priamo stiahnutú v tom čase z hostingu od WladinQ, ktorý tam mal pravdepodobne aj iné testovacie súbory (viď e76e09954e8facf47b6daed2f932fd60b2289846)Máš nejaké ďalšie návrhy na zmazanie?