Hi,

Thanks for the valuable feedback, we will pick up these issues and work on
a new release.

Regards,
Hans

On Sun, Jan 3, 2021 at 2:59 AM Justin Mclean <jus...@classsoftware.com>
wrote:

> HI,
>
> Sorry but it’s -1 (binding) as it contains Category B source code and
> header issues and is not using the WIP disclaimer. [1]
>
>  I suggest you use the work in progress disclaimer, and in it list the
> issues that need to be fixed before graduation. Fixing some of teh header
> issues would make it easier to review as well.
>
> I checked:
> - Incubating in name
> - DISCLAIMER exists but is not work in progress
> - LICENSE is missing mention of 3rd party included software [2][3][5][6]
> - NOTICE may need updating if the release contains code from Apache Hadoop
> - There a large number of files that are missing ASF headers. This makes
> it hard to review.
> - There are a large number of files without correct headers copyright
> Hitachi Vantara.
> - Files incorrectly have the ASF header onto of anther license header e.g
> [4]
> - Includes Category B course code
> - No unexpected binary files
> - Unable to compile on OSX but not sure if that’s a supported platform. It
> gets to step 467 out of 474. (Hop Assemblies Plugins Dist)
>
> Thanks,
> Justin
>
> 1.
> https://incubator.apache.org/policy/incubation.html#work_in_progress_disclaimer
> 2. OpenSansLight font
> 3.
> ./plugins/actions/ftp/src/main/java/org/apache/hop/workflow/actions/ftpput/FtpClient.java
> 4.
> ./ui/src/main/java/org/apache/hop/ui/pipeline/transforms/tableinput/SqlValuesHighlight.java
> 5. ./assemblies/static/src/main/resources/LICENSE ( Copyright (c) 2005,
> European Commission project)
> 6
> ./assemblies/static/src/main/resources/docs/English/welcome/css/bootstrap.css
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Reply via email to