You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sebb <se...@gmail.com> on 2022/06/27 09:34:21 UTC

[ALL] Add *.json to RAT excludes in parent POM ?

JSON files don't support comments, so it's not feasible to add an AL
header to them.

So rather than have to add such excludes at component level, it might
be a good idea to update the parent POM.

Any objections?

Sebb

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [ALL] Add *.json to RAT excludes in parent POM ?

Posted by Matt Sicker <bo...@gmail.com>.
Some JSON parsers do support comments. Some use cases could potentially use YAML or similar. Otherwise, yeah, might as well exclude where comments can’t be added.

—
Matt Sicker

> On Jun 27, 2022, at 08:42, Gilles Sadowski <gi...@gmail.com> wrote:
> 
> Hello.
> 
>> Le lun. 27 juin 2022 à 15:30, Gary Gregory <ga...@gmail.com> a écrit :
>> 
>> Test fixtures in Commons Configuration and SCXML for example.
>> 
>> Visual Code project settings in Commons Daemon
>> 
>> RevApi configuration in Commons RDF.
> 
> Can't some other format be used that doesn't have this shortcoming?
> 
> Gilles
> 
>> 
>> Just search for *.json...
>> 
>> Gary
>> 
>> 
>>> On Mon, Jun 27, 2022, 08:17 Gilles Sadowski <gi...@gmail.com> wrote:
>>> 
>>> Hello.
>>> 
>>> Le lun. 27 juin 2022 à 11:34, sebb <se...@gmail.com> a écrit :
>>>> 
>>>> JSON files don't support comments, so it's not feasible to add an AL
>>>> header to them.
>>> 
>>> What are files in this format used for (in a "Commons" component)?
>>> 
>>> Gilles
>>> 
>>>> 
>>>> So rather than have to add such excludes at component level, it might
>>>> be a good idea to update the parent POM.
>>>> 
>>>> Any objections?
>>>> 
>>>> Sebb
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
> 

Re: [ALL] Add *.json to RAT excludes in parent POM ?

Posted by Gilles Sadowski <gi...@gmail.com>.
Hello.

Le lun. 27 juin 2022 à 15:30, Gary Gregory <ga...@gmail.com> a écrit :
>
> Test fixtures in Commons Configuration and SCXML for example.
>
> Visual Code project settings in Commons Daemon
>
> RevApi configuration in Commons RDF.

Can't some other format be used that doesn't have this shortcoming?

Gilles

>
> Just search for *.json...
>
> Gary
>
>
> On Mon, Jun 27, 2022, 08:17 Gilles Sadowski <gi...@gmail.com> wrote:
>
> > Hello.
> >
> > Le lun. 27 juin 2022 à 11:34, sebb <se...@gmail.com> a écrit :
> > >
> > > JSON files don't support comments, so it's not feasible to add an AL
> > > header to them.
> >
> > What are files in this format used for (in a "Commons" component)?
> >
> > Gilles
> >
> > >
> > > So rather than have to add such excludes at component level, it might
> > > be a good idea to update the parent POM.
> > >
> > > Any objections?
> > >
> > > Sebb

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [ALL] Add *.json to RAT excludes in parent POM ?

Posted by Gary Gregory <ga...@gmail.com>.
Test fixtures in Commons Configuration and SCXML for example.

Visual Code project settings in Commons Daemon

RevApi configuration in Commons RDF.

Just search for *.json...

Gary


On Mon, Jun 27, 2022, 08:17 Gilles Sadowski <gi...@gmail.com> wrote:

> Hello.
>
> Le lun. 27 juin 2022 à 11:34, sebb <se...@gmail.com> a écrit :
> >
> > JSON files don't support comments, so it's not feasible to add an AL
> > header to them.
>
> What are files in this format used for (in a "Commons" component)?
>
> Gilles
>
> >
> > So rather than have to add such excludes at component level, it might
> > be a good idea to update the parent POM.
> >
> > Any objections?
> >
> > Sebb
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [ALL] Add *.json to RAT excludes in parent POM ?

Posted by Gilles Sadowski <gi...@gmail.com>.
Hello.

Le lun. 27 juin 2022 à 11:34, sebb <se...@gmail.com> a écrit :
>
> JSON files don't support comments, so it's not feasible to add an AL
> header to them.

What are files in this format used for (in a "Commons" component)?

Gilles

>
> So rather than have to add such excludes at component level, it might
> be a good idea to update the parent POM.
>
> Any objections?
>
> Sebb

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [ALL] Add *.json to RAT excludes in parent POM ?

Posted by Gary Gregory <ga...@gmail.com>.
This lameness in JSON asks of us to provide this convenience sadly.

Gary

On Mon, Jun 27, 2022, 05:34 sebb <se...@gmail.com> wrote:

> JSON files don't support comments, so it's not feasible to add an AL
> header to them.
>
> So rather than have to add such excludes at component level, it might
> be a good idea to update the parent POM.
>
> Any objections?
>
> Sebb
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>