| 1 |
On Sun, Oct 03, 2021 at 08:58:00AM +0200, Fabian Groffen wrote: |
| 2 |
> On 02-10-2021 23:03:56 -0400, Ionen Wolkens wrote: |
| 3 |
> > On Sat, Oct 02, 2021 at 10:53:37PM -0400, Ionen Wolkens wrote: |
| 4 |
> > > Guess there's a lot of other options that could be considered as well. |
| 5 |
> > > |
| 6 |
> > > --- files |
| 7 |
> > > >>> text |
| 8 |
> > > * current, it wasn't aligned now that I look at it again |
| 9 |
> > > (relying only on color to convey type feels clearly wrong to me) |
| 10 |
> > > |
| 11 |
> > > --- files |
| 12 |
> > > >>>> text |
| 13 |
> > > [QA] new based on current PR |
| 14 |
> > > |
| 15 |
> > > >>> text |
| 16 |
> > > [QA] aligned 1 character further, maybe skipping changing >>> is fine? |
| 17 |
> > > (then again that it's further is what threw me off at first) |
| 18 |
> > > |
| 19 |
> > > >>> text |
| 20 |
> > > QA* similar to before, but aligned using only 3 chars |
| 21 |
> > > |
| 22 |
> > > >>> text |
| 23 |
> > > [Q] kinda more obscure but it can work |
| 24 |
> > > |
| 25 |
> > |
| 26 |
> > Guess should also add these: |
| 27 |
> > >>> text |
| 28 |
> > Q* Notice: |
| 29 |
> > E* Some error happened |
| 30 |
> > (closest to before by making use of the former leading space, thus |
| 31 |
> > no alignment changes) |
| 32 |
> |
| 33 |
> FWIW, I like this one. Perhaps even with lowercase |
| 34 |
> |
| 35 |
> make[4]: leaving directory src |
| 36 |
> q* soname lacks version |
| 37 |
> e* failed to die |
| 38 |
|
| 39 |
Also I guess it provides some degree of compatibility with external |
| 40 |
scripts/tools that adopted the ` * ` format to copy portage. |
| 41 |
i.e. could actually keep ` * ` for einfo, no need for `.* ` here. |
| 42 |
|
| 43 |
Lowercase might be a good idea too, albeit I'm still undecided on |
| 44 |
what I like best in general. |
| 45 |
|
| 46 |
> |
| 47 |
> Fabian |
| 48 |
> |
| 49 |
> > |
| 50 |
> > >>> text |
| 51 |
> > QA Notice: |
| 52 |
> > EE Some error happened |
| 53 |
> > (at least clearer than Q* Notice, but unsure about no separator.. guess |
| 54 |
> > it could work?) |
| 55 |
> > |
| 56 |
> > > >>>> text |
| 57 |
> > > QA* probably closest to how it was before alignment-wise, but meh at 4 > |
| 58 |
> > > |
| 59 |
> > > >>> message |
| 60 |
> > > QA> not convinced about this one, but throwing it here anyway |
| 61 |
> > > (other characters could be considered as well) |
| 62 |
> > > |
| 63 |
> > > Maybe a poll of some kind may help, personally undecided on what I |
| 64 |
> > > like better beside agreeing that a change is needed. |
| 65 |
> > |
| 66 |
> > |
| 67 |
> > -- |
| 68 |
> > ionen |
| 69 |
> |
| 70 |
> |
| 71 |
> |
| 72 |
> -- |
| 73 |
> Fabian Groffen |
| 74 |
> Gentoo on a different level |
| 75 |
|
| 76 |
|
| 77 |
|
| 78 |
-- |
| 79 |
ionen |
Archives