Date: Mon, 30 Jan 2023 22:09:21 +0100 From: Dmitry Salychev <dsl@mcusim.org> To: Warner Losh <imp@bsdimp.com> Cc: Kurt Jaeger <pi@freebsd.org>, Stephane Rochoy <stephane.rochoy@stormshield.eu>, freebsd-current@freebsd.org Subject: Re: Tooling Integration and Developer Experience Message-ID: <86sffrohfn.fsf@peasant.tower.home> In-Reply-To: <CANCZdfrpx9MW0wE0oWnkk5LmqbMVVaC_2z5fzaidK4QnRG2eTg@mail.gmail.com> References: <202301300254.30U2sm0k061914@dell.no.berklix.net> <97020cad-f913-2985-2093-e4c23bf671e3@antonovs.family> <86357sxsly.fsf@cthulhu.stephaner.labo.int> <Y9eepV%2B%2Bch6qMBta@home.opsec.eu> <86edrcp13z.fsf@peasant.tower.home> <CANCZdfp7RpYyGH1-yx-ddaibwM3=sfYOYw8m7mKjJ4gfWjUW6Q@mail.gmail.com> <86zga0nf1q.fsf@peasant.tower.home> <CANCZdfrpx9MW0wE0oWnkk5LmqbMVVaC_2z5fzaidK4QnRG2eTg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Maybe, but my laziness won't let me keep it that simple :) https://reviews.freebsd.org/H185 should do the trick, I think. Warner Losh <imp@bsdimp.com> writes: > I think just "sys/dev/dpaa2" is all you need when adding the Herald rule. :) > > More generally, no, there's no way that I've found to only match files that match certain regexp. > > Warner > > On Mon, Jan 30, 2023 at 9:49 AM Dmitry Salychev <dsl@mcusim.org> wrote: > > Warner Losh <imp@bsdimp.com> writes: > > > On Mon, Jan 30, 2023 at 7:07 AM Dmitry Salychev <dsl@mcusim.org> wrote: > > > > Hi, > > > > Kurt Jaeger <pi@freebsd.org> writes: > > > > > Hi, > > > > > >> > On 1/30/23 02:54, Julian H. Stacey wrote: > > >> > The main idea: to prevent information fragmentation and improve > > >> > discoverability, cross-referencing abilities, search, etc. > > >> > > >> With regards to improving discoverability, Phabricator's Owner > > >> tool could be a good tactical move: it allow to bind code area to > > >> peoples in order to automatically add them to reviews. > > > > > > If you know phabricator in more detail, is there any kind of tool > > > to understand the activity going on ? > > > > > > In bugs.freebsd.org, there is the dashboard: > > > > > > https://bugs.freebsd.org/bugzilla/page.cgi?id=dashboard.html > > > > > > I think we might need something similar to help us understand > > > the current state of the phabricator instance and the work > > > being done. > > > > > > Phab allows Dashboards, but no-one had the time to configure some > > > queries to provide relevant stats. > > > > I'd prefer to have an automatic note via email from Phabricator if any > > file with my copyright was modified/patched/improved in a newly opened > > review. > > > > I don't think that people tend to ignore changes to those parts of code > > they've written (or touched at least). > > > > You can easily create a herald rule to do this. I have one for all of src, but others have subsets > > like the boot loader. And anybody with an account can do that. > > > > Warner > > Looks really useful, thanks for a hint. Is there any way to test regexp > matching for files content? Btw, there could be a carefully selected set > of pre-defined Herald rules added to any new committer's account. > > -- > Dmitry Salychev -- Dmitry Salychev
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86sffrohfn.fsf>