Re: [PATCH V2] scripts/checkpatch.pl: check for non-permalinks to Zulip

From: Miguel Ojeda
Date: Wed Aug 28 2024 - 05:27:31 EST


On Tue, Aug 27, 2024 at 8:44 PM Siddharth Menon <simeddon@xxxxxxxxx> wrote:
>
> Zulip links to https://rust-for-linux.zulipchat.com can break in
> case of renaming the topic or channel if they are not message
> links (which are permanent links).
>
> If a non-permanent Zulip link is referenced then emit a warning
> and direct the user to the Zulip documentation.
>
> Permanent links are of the format:
> https://.../#narrow/stream/x/topic/x/near/<numerical_id>
>
> Suggested-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
> Link: https://github.com/Rust-for-Linux/linux/issues/1104
> Signed-off-by: Siddharth Menon <simeddon@xxxxxxxxx>

This looks better, thanks!

> + "Use permanent Zulip links when possible - see https://zulip.com/help/link-to-a-message-or-conversation\n"; . $herecurr);

That link is better, yeah.

It may still be a bit confusing (one needs to search for "permalink"
inside the page), but perhaps Zulip will improve those when they
introduce more permalinks (?).

I guess we could link directly to the section at
https://zulip.com/help/link-to-a-message-or-conversation#get-a-link-to-a-specific-message
-- it may help a bit.

We could also explain how to shorten the links if the line is very
long, but that is probably best left for another patch (possibly
another warning).

Cheers,
Miguel