Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-14 Thread Alexey Zinoviev
Ok, thank you, I've missed the start of the discussion пн, 14 окт. 2019 г. в 11:13, Ivan Pavlukhin : > Alexey, Petr, > > Please read first message in this thread. Now checkstyle is a part of > [Build > Apache Ignite]. > > If you still have any questions after that please write back. > Sincerely,

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-14 Thread Ivan Pavlukhin
Alexey, Petr, Please read first message in this thread. Now checkstyle is a part of [Build Apache Ignite]. If you still have any questions after that please write back. Sincerely, I do not want us to repeat the discussion as it was detailed enough and in my opinion Maxim did the change gracefully

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-14 Thread Petr Ivanov
Maksim, why have you removed Checkstyle from Run All? > On 13 Oct 2019, at 18:52, Maxim Muzafarov wrote: > > Igniters, > > The `checkstyle` profile has been enabled for the Apache Ignite build > procedure. > Unused imports in the spark module have been fixed [1]. > The `[Check Code Style]` su

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-13 Thread Alexey Zinoviev
Please, return checkstyle to run Ml, I use this feature вс, 13 окт. 2019 г., 18:52 Maxim Muzafarov : > Igniters, > > The `checkstyle` profile has been enabled for the Apache Ignite build > procedure. > Unused imports in the spark module have been fixed [1]. > The `[Check Code Style]` suite has be

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-13 Thread Maxim Muzafarov
Igniters, The `checkstyle` profile has been enabled for the Apache Ignite build procedure. Unused imports in the spark module have been fixed [1]. The `[Check Code Style]` suite has been excluded from `Run :: All`, `Run :: ML`. [1] https://issues.apache.org/jira/browse/IGNITE-12287 On Tue, 8 Oct

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-07 Thread Ivan Pavlukhin
Let's give it a try. пн, 7 окт. 2019 г. в 13:21, Nikolay Izhikov : > > +1 > > В Пн, 07/10/2019 в 13:18 +0300, Maxim Muzafarov пишет: > > Igniters, > > > > > > I'm planning October 11 (Friday) 22-00 MSK enable `checkstyle` profile > > on the [Build > > Apache Ignite] suite by the end of the next we

Re: Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-07 Thread Nikolay Izhikov
+1 В Пн, 07/10/2019 в 13:18 +0300, Maxim Muzafarov пишет: > Igniters, > > > I'm planning October 11 (Friday) 22-00 MSK enable `checkstyle` profile > on the [Build > Apache Ignite] suite by the end of the next weekend (one week test > period). Such an option discussed many times before (e.g. [1])

Enabling the checkstyle profile on Build Apache Ignite suite (test period)

2019-10-07 Thread Maxim Muzafarov
Igniters, I'm planning October 11 (Friday) 22-00 MSK enable `checkstyle` profile on the [Build Apache Ignite] suite by the end of the next weekend (one week test period). Such an option discussed many times before (e.g. [1]). Here are the reasons: - any code style violations in a PR lead to sou