Hi Shiyan,
Thanks for your awesome work!
Will follow you and try to get further work done!
Best,
Vino
Shiyan Xu 于2020年4月24日周五 上午9:24写道:
> Also thank @vinoyang for taking on the reviews!
>
> > [Wondering if there is a way to stick a checkstyle rule to this effect.
> guess it won't check for
Also thank @vinoyang for taking on the reviews!
> [Wondering if there is a way to stick a checkstyle rule to this effect.
guess it won't check for new changes alone, rather complain about existing
junit 4 tests?]
Yes checkstyle complains all... will add the rule after API migration.
On Thu, Apr
Good job Raymond ! and thanks for the reminder.
On Wed, Apr 22, 2020 at 11:42 AM leesf wrote:
> Thanks for the reminder, I upgraded the to junit5 for the PR
> https://github.com/apache/incubator-hudi/pull/1536 and will take an eye on
> when reviewing PRs.
>
> Bhavani Sudha 于2020年4月22日周三
Thanks for the reminder, I upgraded the to junit5 for the PR
https://github.com/apache/incubator-hudi/pull/1536 and will take an eye on
when reviewing PRs.
Bhavani Sudha 于2020年4月22日周三 下午3:31写道:
> +1. Thanks for the update Raymond and great work on the migration.
>
> -Sudha
>
> On Tue, Apr 21,
+1. Thanks for the update Raymond and great work on the migration.
-Sudha
On Tue, Apr 21, 2020 at 10:39 PM Vinoth Chandar wrote:
> +1 Appreciate the efforts, Raymond!
>
> [Wondering if there is a way to stick a checkstyle rule to this effect.
> guess it won't check for new changes alone,
+1 Appreciate the efforts, Raymond!
[Wondering if there is a way to stick a checkstyle rule to this effect.
guess it won't check for new changes alone, rather complain about existing
junit 4 tests?]
On Tue, Apr 21, 2020 at 5:10 PM Shiyan Xu
wrote:
> Hi all,
>
> We're in progress with JUnit 5
Hi all,
We're in progress with JUnit 5 migration for all test classes. So far the
JUnit 5 dependencies (including Mockito) have been added to all modules.
The APIs/modules migration status is shown here
https://github.com/apache/incubator-hudi/pull/1530#issue-405575235
I would like to kindly ask