Re: [PATCH 4/6] [GSOC] ref-filter: add %(rest) atom and --rest option

From: ZheNing Hu <adlternative@gmail.com>
Date: Jun 8, 2021, 06:16 UTC
Message-ID: <CAOLTT8TSue=Cx8xos20vnGSi3oOd8+=jTfTw2h82gXmmd4KyLg@mail.gmail.com>
In-Reply-To: <CAOLTT8QE7pafPmhnz-6=5zuyjg9n1FNbu_k6bA80jE1e5vYCmQ@mail.gmail.com>

ZheNing Hu <adlternative@gmail.com> 于2021年6月7日周一 下午9:18写道：

Show 12 quoted lines

>
> ZheNing Hu <adlternative@gmail.com> 于2021年6月7日周一 下午9:02写道：
> >
> > Hope we can reach an agreement:
> > delete `--rest` and add `--reject-atoms`. ;-)
> >
>
> I forget one thing: %(raw:textconv) and %(raw:filters) can use
> the value of "--rest" as their <path>. But now if we want delete --rest,
> they can not be used for "for-each-ref" family, Git will die with
> "missing path for 'xxx'".
>

If we actually delete "--rest", we will have no way to test %(raw:textconv) and %(raw:filters)... So now I think we can keep --rest (or use another name --path) and let "git for-each-ref" family reject %(rest) by default.

Thanks. -- ZheNing Hu