Page MenuHomeFreeBSD

Commit message template: PR field now accepts bugs.freebsd.org URL.
AcceptedPublic

Authored by lwhsu on Dec 23 2022, 6:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 17 2024, 5:17 AM
Unknown Object (File)
Oct 16 2024, 2:12 AM
Unknown Object (File)
Sep 15 2024, 12:51 AM
Unknown Object (File)
Sep 6 2024, 3:58 AM
Unknown Object (File)
Aug 31 2024, 7:21 AM
Unknown Object (File)
Aug 31 2024, 7:21 AM
Unknown Object (File)
Aug 31 2024, 7:06 AM
Unknown Object (File)
Aug 12 2024, 8:30 AM

Details

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 48874
Build 45763: arc lint + arc unit

Event Timeline

lwhsu requested review of this revision.Dec 23 2022, 6:18 PM
zlei added a subscriber: zlei.

Looks good to me.

This revision is now accepted and ready to land.Dec 25 2022, 10:14 AM
grahamperrin added inline comments.
tools/tools/git/hooks/prepare-commit-msg
45

Is it clear enough that there should be no more than one URL per line?

A committer might type something like this (not exceeding 72 characters):

PR: https://bugs.freebsd.org/###### https://bugs.freebsd.org/######

tools/tools/git/hooks/prepare-commit-msg
45

… sorry, I could not find the related report at the time of my previous comment.

I visualise any string of two or more white space-separated URLs, on a single line, resulting in a single, invalid link in cgit.

From FreeBSD bug 266473 – cgit.freebsd.org breaks some links – seems to not recognise some types of URI delimiter (RFC 3986):

{F53896353}