* Re: [PATCH 4/3] install/README: improve wording
2023-10-17 7:54 6% ` Eric Wong
@ 2023-10-17 8:07 6% ` Štěpán Němec
0 siblings, 0 replies; 4+ results
From: Štěpán Němec @ 2023-10-17 8:07 UTC (permalink / raw)
To: Eric Wong; +Cc: meta
On Tue, 17 Oct 2023 07:54:28 +0000
Eric Wong wrote:
>> > -* users using from git or tarballs (and not the OS package manager)
>> > +* users running our code from git or tarballs (and not the OS package manager)
>>
>> LGTM, but I'd also wrap the line (the text file line length
>> norm in the repo seems to lie between 60 and 70; this would
>> be 78).
>
> I think it's fine for a standalone lines such as list items to
> be up to 80. While paragraphs get harder to read with more
> columns, standalone lines get distracting when wrapped (IMHO).
I'd still at least make a distinction between "true" lists
such as the package listings in INSTALL (which are also
columnized, so the actual "line" is quite short), and what's
closer to just a sentence with a bullet point, like here.
In the given context I don't find
* users running our code from git or tarballs (and not the
OS package manager)
distracting, unlike the overlong line, but it's your call of
course.
--
Štěpán
^ permalink raw reply [relevance 6%]
* Re: [PATCH 4/3] install/README: improve wording
2023-10-17 7:19 6% ` Štěpán Němec
@ 2023-10-17 7:54 6% ` Eric Wong
2023-10-17 8:07 6% ` Štěpán Němec
0 siblings, 1 reply; 4+ results
From: Eric Wong @ 2023-10-17 7:54 UTC (permalink / raw)
To: Štěpán Němec; +Cc: meta
Štěpán Němec <stepnem@smrk.net> wrote:
> On Tue, 17 Oct 2023 06:43:18 +0000
> Eric Wong wrote:
>
> > -* users using from git or tarballs (and not the OS package manager)
> > +* users running our code from git or tarballs (and not the OS package manager)
>
> LGTM, but I'd also wrap the line (the text file line length
> norm in the repo seems to lie between 60 and 70; this would
> be 78).
I think it's fine for a standalone lines such as list items to
be up to 80. While paragraphs get harder to read with more
columns, standalone lines get distracting when wrapped (IMHO).
^ permalink raw reply [relevance 6%]
* Re: [PATCH 4/3] install/README: improve wording
2023-10-17 6:43 14% ` [PATCH 4/3] install/README: improve wording Eric Wong
@ 2023-10-17 7:19 6% ` Štěpán Němec
2023-10-17 7:54 6% ` Eric Wong
0 siblings, 1 reply; 4+ results
From: Štěpán Němec @ 2023-10-17 7:19 UTC (permalink / raw)
To: Eric Wong; +Cc: meta
On Tue, 17 Oct 2023 06:43:18 +0000
Eric Wong wrote:
> -* users using from git or tarballs (and not the OS package manager)
> +* users running our code from git or tarballs (and not the OS package manager)
LGTM, but I'd also wrap the line (the text file line length
norm in the repo seems to lie between 60 and 70; this would
be 78).
Thanks,
Štěpán
^ permalink raw reply [relevance 6%]
* [PATCH 4/3] install/README: improve wording
@ 2023-10-17 6:43 14% ` Eric Wong
2023-10-17 7:19 6% ` Štěpán Němec
0 siblings, 1 reply; 4+ results
From: Eric Wong @ 2023-10-17 6:43 UTC (permalink / raw)
To: Štěpán Němec; +Cc: meta
Štěpán Němec <stepnem@smrk.net> wrote:
> https://grammarhow.com/or-vs-nor/
Thanks, that's better than the results that were at the top for me.
> >> Files in this directory are designed for:
> >>
> >> -* users using from git or tarballs (and not the OS package manager)
> >> +* users using git repo or tarballs (and not the OS package manager)
> >
> > I wonder if:
> >
> > * users using our code from git or tarballs (and not the OS package manager)
> >
> > ...is better. I always feel the need to clarify git the software
> > vs. our code repo vs. our mirror on public-inbox.org/git *shrug*
>
> Yeah, I agree that's better. Maybe also change "using" to
> "running"? "Users using" is a bit awkward.
Agreed. How about:
----------8<---------
Subject: [PATCH 4/3] install/README: improve wording
Reported-by: Štěpán Němec <stepnem@smrk.net>
---
install/README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/install/README b/install/README
index 981e014b..dd652552 100644
--- a/install/README
+++ b/install/README
@@ -5,7 +5,7 @@ TL;DR: sudo ./install/deps.perl all
Files in this directory are designed for:
-* users using from git or tarballs (and not the OS package manager)
+* users running our code from git or tarballs (and not the OS package manager)
* lazy users who can't be bothered to read all of INSTALL
^ permalink raw reply related [relevance 14%]
Results 1-4 of 4 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2023-10-16 11:33 [PATCH 1/3] doc: fix some typos and grammar Štěpán Němec
2023-10-16 21:13 ` Eric Wong
2023-10-16 22:16 ` Štěpán Němec
2023-10-17 6:43 14% ` [PATCH 4/3] install/README: improve wording Eric Wong
2023-10-17 7:19 6% ` Štěpán Němec
2023-10-17 7:54 6% ` Eric Wong
2023-10-17 8:07 6% ` Štěpán Němec
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).