Re: [PATCH v2] rust: MAINTAINERS: Add the zulip link

From: Miguel Ojeda
Date: Tue Jan 31 2023 - 20:38:45 EST


On Sat, Jan 28, 2023 at 8:23 AM Boqun Feng <boqun.feng@xxxxxxxxx> wrote:
>
> Zulip organization "rust-for-linux" has been created since about 2 years

"was created about 2 years ago ... and has proven" maybe?

> [1]: https://lore.kernel.org/rust-for-linux/CANiq72=xVaMQkgCA9rspjV8bhWDGqAn4x78B0_4U1WBJYj1PiA@xxxxxxxxxxxxxx/

We typically do links as a tag, like:

Link: https://..../ [1]

> R: Gary Guo <gary@xxxxxxxxxxx>
> R: Björn Roy Baron <bjorn3_gh@xxxxxxxxxxxxxx>
> L: rust-for-linux@xxxxxxxxxxxxxxx
> +C: zulip://rust-for-linux.zulipchat.com
> S: Supported
> W: https://github.com/Rust-for-Linux/linux
> B: https://github.com/Rust-for-Linux/linux/issues

The preferred order according to the top of the file is `C:` after
`B:` -- could you please move it? (even if not everybody follows it)

Thanks Boqun!

Also, for reference for others: a few of us discussed a bit in Zulip
whether to use `zulip://` or `https://` here: `zulip://` hints at the
fact that it is possible to join with apps/clients and not just with a
web browser, which is why Boqun wants to use this one. Support for the
URI scheme in place of the `https://` one isn't great in some of the
apps, though, so I worried it could be confusing.

Hopefully the URI scheme support will improve in the future. So let's
try `zulip://` -- it may help getting better support for that URI
scheme too :)

Cheers,
Miguel