+1 to your proposal. I added a review comment in two of the PRs.
On Fri, Jul 27, 2018 at 9:06 AM Chris Brody <chris.br...@gmail.com> wrote:
>
> While getting ready to make a cordova-windows patch release I
> discovered that coho audit-license-headers flags an issue with
> template/Properties/Default.rd.xml ([1]). Proposed solution is to add
> the needed license header to template/Properties/Default.rd.xml in the
> patch release as proposed in [2] then fix in upcoming release as
> proposed in [3,4]. Alternatives are discussed in [5].
>
> I would personally favor fixing as proposed in [2,3,4] and consider if
> we can remove this file as discussed in [5].
>
> Feedback in [5] or this thread would be really appreciated. I would
> like to make the cordova-windows patch release within the next few
> days, hopefully by the end of July 2018.
>
> Thanks and best regards,
>
> Chris
>
> [1] <https://issues.apache.org/jira/browse/CB-14224>
> [2] <https://github.com/apache/cordova-windows/pull/286>
> [3] <https://github.com/apache/cordova-windows/pull/284>
> [4] <https://github.com/apache/cordova-windows/pull/285>
> [5] <https://github.com/apache/cordova-discuss/issues/106>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
> For additional commands, e-mail: dev-h...@cordova.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Reply via email to