Disregard. You’re right. webpack config is .js. Missed that.

You see any issues with our disclaimer given that the webpack config doesn’t 
have a header?

Joshua Poore


> On Feb 10, 2020, at 7:32 PM, Joshua Poore <[email protected]> wrote:
> 
> Hi Dave,
> 
> Thanks for identifying! I thought comments aren’t supported in JSON. There 
> are custom libs for doing that, but I doubt node and web pack can call them. 
> 
> If you can point me to an example, I’ll get it in there.
> 
> Joshua Poore
> 
> 
>>>> On Feb 10, 2020, at 5:03 PM, David Meikle <[email protected]> wrote:
>>>> On Fri, 7 Feb 2020 at 23:12, Joshua Poore <[email protected]> wrote:
>>>> Hi Folks,
>>>> Please VOTE on the Apache Flagon 2.1.0 Release Candidate 01
>>>> Source Release Artifacts:
>>>> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.1.0-RC-01/
>>>> <
>>>> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.1.0-RC-01/
>>>> Vote will be open for 72 hours. Please VOTE as follows:
>>>> [ ] +1, let's get it released!!!
>>>> [ ] +/-0, fine, but consider to fix few issues before...
>>>> [ ] -1, nope, because... (and please explain why)
>> +1 (binding) : Performed the check list provided on the source release
>> package.
>> There are some files, like the webpack.config.js, flagged by Apache RAT
>> that we could put an AL header on. Not a deal breaker, but worth
>> considering.
>> Cheers,
>> Dave

Reply via email to