On Sun, Oct 03, 2021 at 08:58:00AM +0200, Fabian Groffen wrote: > On 02-10-2021 23:03:56 -0400, Ionen Wolkens wrote: > > On Sat, Oct 02, 2021 at 10:53:37PM -0400, Ionen Wolkens wrote: > > > Guess there's a lot of other options that could be considered as well. > > > > > > --- files > > > >>> text > > > * current, it wasn't aligned now that I look at it again > > > (relying only on color to convey type feels clearly wrong to me) > > > > > > --- files > > > >>>> text > > > [QA] new based on current PR > > > > > > >>> text > > > [QA] aligned 1 character further, maybe skipping changing >>> is fine? > > > (then again that it's further is what threw me off at first) > > > > > > >>> text > > > QA* similar to before, but aligned using only 3 chars > > > > > > >>> text > > > [Q] kinda more obscure but it can work > > > > > > > Guess should also add these: > > >>> text > > Q* Notice: > > E* Some error happened > > (closest to before by making use of the former leading space, thus > > no alignment changes) > > FWIW, I like this one. Perhaps even with lowercase > > make[4]: leaving directory src > q* soname lacks version > e* failed to die
Also I guess it provides some degree of compatibility with external scripts/tools that adopted the ` * ` format to copy portage. i.e. could actually keep ` * ` for einfo, no need for `.* ` here. Lowercase might be a good idea too, albeit I'm still undecided on what I like best in general. > > Fabian > > > > > >>> text > > QA Notice: > > EE Some error happened > > (at least clearer than Q* Notice, but unsure about no separator.. guess > > it could work?) > > > > > >>>> text > > > QA* probably closest to how it was before alignment-wise, but meh at 4 > > > > > > > >>> message > > > QA> not convinced about this one, but throwing it here anyway > > > (other characters could be considered as well) > > > > > > Maybe a poll of some kind may help, personally undecided on what I > > > like better beside agreeing that a change is needed. > > > > > > -- > > ionen > > > > -- > Fabian Groffen > Gentoo on a different level -- ionen
signature.asc
Description: PGP signature
