From: Eli Zaretskii <eliz@gnu.org>
To: Sean Whitton <spwhitton@spwhitton.name>
Cc: stefankangas@gmail.com, arsen@aarsen.me, Hi-Angel@yandex.ru,
emacs-devel@gnu.org, jporterbugs@gmail.com
Subject: Re: Adding git-commit highlight mode?
Date: Sat, 04 Jan 2025 09:17:06 +0200 [thread overview]
Message-ID: <86o70nf3pp.fsf@gnu.org> (raw)
In-Reply-To: <87ed1j8vc2.fsf@zephyr.silentflame.com> (message from Sean Whitton on Fri, 03 Jan 2025 21:02:21 +0000)
> From: Sean Whitton <spwhitton@spwhitton.name>
> Date: Fri, 03 Jan 2025 21:02:21 +0000
>
> Hello,
>
> On Thu 02 Jan 2025 at 03:53pm -06, Stefan Kangas wrote:
>
> > AFAICT, the 50 character line limit for the summary line is not really
> > followed very strictly in practice by most projects, including the Linux
> > kernel. It's more of a recommendation. Even the link you provide above
> > says that it's "a good idea" to use "no more than 50 characters" for it.
> > For that reason, I think that this should be a warning (with a highlight
> > if you go over), and not a hard limit.
> >
> > In `git-commit-mode`, the user option `git-commit-summary-max-length' is
> > 68 by default, which strikes me as unopinionated enough.
>
> As Jonas mentioned, some of the things suggested in this thread overlap
> with our existing vc-git-log-edit-mode. For example, we already have
> vc-git-log-edit-summary-target-len and vc-git-log-edit-summary-max-len.
vc-git-log-edit-mode kicks in only if one commits using VC commands.
It will not kick in if, for example, one commits from the shell prompt
with GIT_EDITOR set to invoke emacsclient.
So there's still a place for a mode that is turned on when Emacs needs
to edit the files with special names used by Git for some of the
activities.
> (As the VC maintainer, I would have appreciated being CC'd on this
> thread at some point; glad I caught it anyway.)
The thread was not about VC (still isn't, AFAICT), so it is not
surprising you were not CC'ed.
next prev parent reply other threads:[~2025-01-04 7:17 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-02 18:30 Adding git-commit highlight mode? Konstantin Kharlamov
2025-01-02 19:01 ` Eli Zaretskii
2025-01-02 19:07 ` Konstantin Kharlamov
2025-01-02 20:10 ` Stefan Kangas
2025-01-02 21:01 ` Eli Zaretskii
2025-01-02 21:38 ` Stefan Kangas
2025-01-03 5:26 ` Jim Porter
2025-01-02 21:40 ` Konstantin Kharlamov
2025-01-03 5:29 ` Jim Porter
2025-01-03 13:08 ` Jonas Bernoulli
2025-01-02 20:11 ` Eli Zaretskii
2025-01-02 21:19 ` Arsen Arsenović
2025-01-02 21:53 ` Stefan Kangas
2025-01-02 22:27 ` Arsen Arsenović
2025-01-03 21:02 ` Sean Whitton
2025-01-04 7:17 ` Eli Zaretskii [this message]
2025-01-04 9:52 ` Sean Whitton
2025-01-03 6:45 ` Eli Zaretskii
2025-01-02 19:17 ` Jim Porter
2025-01-02 19:19 ` Konstantin Kharlamov
2025-01-02 20:17 ` Eli Zaretskii
2025-01-03 21:14 ` Björn Bidar
2025-01-05 14:39 ` Philip Kaludercic
2025-01-05 22:48 ` Konstantin Kharlamov
[not found] ` <87r05jbnw2.fsf@>
2025-01-03 22:52 ` Konstantin Kharlamov
2025-01-04 1:22 ` Björn Bidar
[not found] ` <87v7uv9xub.fsf@>
2025-01-04 1:45 ` Konstantin Kharlamov
2025-01-05 0:46 ` Björn Bidar
[not found] ` <871pxiulxz.fsf@>
2025-01-05 3:55 ` Konstantin Kharlamov
2025-01-04 12:50 ` Jonas Bernoulli
2025-01-04 17:20 ` Jonas Bernoulli
2025-01-05 0:32 ` Björn Bidar
2025-01-05 18:47 ` Jim Porter
2025-01-05 0:42 ` Björn Bidar
[not found] ` <875xmuum48.fsf@>
2025-01-05 3:49 ` Konstantin Kharlamov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86o70nf3pp.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=Hi-Angel@yandex.ru \
--cc=arsen@aarsen.me \
--cc=emacs-devel@gnu.org \
--cc=jporterbugs@gmail.com \
--cc=spwhitton@spwhitton.name \
--cc=stefankangas@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.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).