Re: [PATCH 2/2] docs: Improve wording in Sphinx dependency checker
From: Dongliang Mu
Date: Thu Oct 29 2026 - 08:51:17 EST
On 10/28/26 8:07 AM, Randy Dunlap wrote:
Hi,
On 10/27/26 10:07 AM, Dongliang Mu wrote:
@@ -784,10 +782,10 @@ class SphinxDependencyChecker(MissingCheckers):I think that "are" is correct here since the subject is plural.
if not self.distro_msg:
self.distro_msg = \
"Note: RHEL-based distros typically require extra repositories.\n" \
- "For most, enabling epel and crb are enough:\n" \
+ "For most, enabling EPEL and CRB is enough:\n" \
From my understanding, enabling XXX is the subject, so I change it to "is". However, this is not very clear.
How about "For most, the EPEL and CRB repositories are enough:\n"? This seems more clear.
Dongliang Mu
Otherwise this is a nice improvement. Thanks.
Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>