[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: info.js demo on Texinfo manual updated on website
From: |
Per Bothner |
Subject: |
Re: info.js demo on Texinfo manual updated on website |
Date: |
Tue, 29 Nov 2022 13:30:22 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 |
On 11/29/22 13:18, Gavin Smith wrote:
I have looked at the diff and found which parts of the code were
changed. Can you edit the following to describe the changes, and
send it back, or commit it yourself with a ChangeLog entry,
I'll do that.
One question that occured to be: Would it be clearer to use
semi-colons (instead of commas) to separate entries/sub-entries? I.e.
instead of:
text = text0 + ", " + text;
do:
text = text0 + "; " + text;
I think would be clearer, especially if index entries contain commas.
using the ChangeLog entry as the commit message.
I believe best practice is for the first line of a commit message
to be a self-contained short summary, followed by an empty line
and further details (if any). So some editing of a ChangeLog entry
is usually appropriate before using it as a commit message.
--
--Per Bothner
per@bothner.com http://per.bothner.com/
- Re: info.js demo on Texinfo manual updated on website, (continued)
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/18
- Re: info.js demo on Texinfo manual updated on website, Patrice Dumas, 2022/11/19
- Re: info.js demo on Texinfo manual updated on website, Gavin Smith, 2022/11/27
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/27
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/27
- Re: info.js demo on Texinfo manual updated on website, Gavin Smith, 2022/11/28
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/29
- Re: info.js demo on Texinfo manual updated on website, Gavin Smith, 2022/11/29
- Re: info.js demo on Texinfo manual updated on website,
Per Bothner <=
- Re: info.js demo on Texinfo manual updated on website, Gavin Smith, 2022/11/29
- Re: info.js demo on Texinfo manual updated on website, Gavin Smith, 2022/11/30
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/30
- Re: info.js demo on Texinfo manual updated on website, Per Bothner, 2022/11/27
Re: Status of texinfo/js/yarn.lock ?, Per Bothner, 2022/11/13