Re: Tooling Integration and Developer Experience

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