Re: Reformat method synopsis whitespaces

From: Date: Tue, 01 Jun 2021 21:23:48 +0000
Subject: Re: Reformat method synopsis whitespaces
References: 1 2  Groups: php.doc 
Request: Send a blank email to phpdoc+get-969387949@lists.php.net to get a copy of this message
Hi Kamil, There's an example for 0 parameters in the PR ( https://user-images.githubusercontent.com/6057627/120331916-8c7ded00-c2ee-11eb-9c2b-65fb2dc96449.png ), and my implementation doesn't special-case functions/methods with a single parameter, so these parameters also end up on a new line. Regards: Máté Kamil Tekiela <tekiela246@gmail.com> ezt írta (időpont: 2021. jún. 1., K, 18:49): > Hi Mate, > > What would it look like on a signature with none or a single parameter? > > Regards, > Kamil > > On Tue, 1 Jun 2021 at 17:14, Máté Kocsis <kocsismate90@gmail.com> wrote: > >> Hi Everyone, >> >> I'm looking for feedback with regard to my PR which reformats method >> synopses (https://github.com/php/phd/pull/38): >> >> Currently, we display function/method signatures without any line breaks, >> but with a lot of unnecessary spaces between parameters, as well as around >> the parameter list. See an example here: >> https://pasteboard.co/K4BXRuv.png >> >> My PR would add a line break for each parameter as well as remove any >> other unnecessary whitespaces. See >> https://pasteboard.co/K4BZNN9.png for >> example (but the linked PR has a few more images attached). >> >> I'm curious if anybody sees any issue with the new formatting? Class >> synopses look a bit weird, but I don't think it's worse than what we >> currently have. >> >> Regards: >> Máté >> >

« previous php.doc (#969387949) next »