unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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.



  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).