Folks, I think we can make a small step further with Ignite Inspections.
I've created these tickets [1], [2] for myself according to previously added `idea/ignite_inspections.xml` and I plan to complete them. Who will help me with review and merge? [1] https://issues.apache.org/jira/browse/IGNITE-9311 - Add missing @Override annotation [2] https://issues.apache.org/jira/browse/IGNITE-9312 - Remove unnecessary @SuppressWarnings annotation On Thu, 16 Aug 2018 at 19:53 Dmitriy Pavlov <dpavlov....@gmail.com> wrote: > Hi Pavel, > > Thank you for noticing and bringing it here. I've fixed TC failure. > > Sincerely, > Dmitriy Pavlov > > чт, 16 авг. 2018 г. в 0:10, Pavel Pereslegin <xxt...@gmail.com>: > > > Hello Igniters. > > > > It seems that "idea/ignite_inspections.xml" should be excluded from > > "check-licenses" maven profile, because "_Licenses Headers_" > > configuration always fails now [1] on TeamCity. > > > > [1] > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_LicensesHeaders&tab=buildTypeHistoryList&branch_IgniteTests24Java8=%3Cdefault%3E > > ср, 15 авг. 2018 г. в 20:49, Dmitriy Pavlov <dpavlov....@gmail.com>: > > > > > > I've updated wiki page > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-C.CodeInspection > > > with > > > reference to settings.xml placement in the project. > > > > > > It is only advice, so I hope you don't mind having this reference. > > > > > > ср, 15 авг. 2018 г. в 16:45, Dmitriy Pavlov <dpavlov....@gmail.com>: > > > > > > > Hi Maxim, > > > > > > > > Thank you for stepping in. I've committed the first version here > > > > 'idea/ignite_inspections.xml'. We can move it to project default > later > > when > > > > all inspection problems are fixed. > > > > Commit: > > > > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=ignite.git;a=commit;h=3e0f04edf7cc0aa1631fbd1b9af1e9b87b697eb1 > > > > > > > > > > > > Igniters, > > > > > > > > you can enable this profile using the following steps: Inspections > > > > (icon)->Configure inspections->(settings button)->Import > > Profile->select > > > > file and import. > > > > > > > > Sincerely, > > > > Dmitriy Pavlov > > > > > > > > вт, 14 авг. 2018 г. в 16:31, Maxim Muzafarov <maxmu...@gmail.com>: > > > > > > > >> Dmitry and other Igniters, > > > >> > > > >> Previously you has suggested to commit `Code Inspections` into > Ignite > > > >> codebase. > > > >> It makes sense for me. I think it's the easiest way to share this > > profile > > > >> among community > > > >> members and this inspection can be set as for the project level. > > > >> So, I suggest: > > > >> > > > >> 1) According to Jetbrains documentation [1] the inspection profile > > can be > > > >> placed to > > > >> `<project>/.idea/inspectionProfiles` with name `Project_Default.xml` > > > >> (hope most of us using IDEA for development) > > > >> This allows enable this profile automatically on per project level > and > > > >> will > > > >> simplify > > > >> development process according to rules accepted by our community. > > > >> > > > >> 2) I can file tickets and do some of them to fix inspection failures > > which > > > >> Alexey mentioned > > > >> earlier. Hope other members of community will help me with it. > > > >> > > > >> 3) I think `Inspections (Core)` TeamCity can be triggered as nightly > > build > > > >> as it takes more > > > >> than 4 hours. Suppose, inspection build in each PR is not the best > > way in > > > >> our case. New run > > > >> here [2]. > > > >> > > > >> 4) We can tune our MTCGA.Bot to notify members with new inspection > > > >> failures > > > >> added by them. > > > >> > > > >> > > > >> Also, I've taken Alexey's inspection file as an example, I’ve > checked > > what > > > >> we already discussed > > > >> previously (e.g. `Anonymous class can be converted to lambda` should > > be > > > >> disabled by default) > > > >> and added these additional rules to it: > > > >> - General | Line is longer than allowed by code style > > > >> - Java | Code maturity | Call to 'printStackTrace()' > > > >> - Java | Code style issues | Unnecessary 'null' check before > > 'equals()' > > > >> call > > > >> > > > >> If we decide to proceed I will attach this file to JIRA. > > > >> > > > >> > > > >> [1] https://www.jetbrains.com/help/idea/code-inspection.html > > > >> [2] > > > >> > > > >> > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_InspectionsCore&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=pull%2F3710%2Fhead > > > >> > > > >> > > > >> On Tue, 14 Aug 2018 at 16:19 Dmitriy Pavlov <dpavlov....@gmail.com> > > > >> wrote: > > > >> > > > >> > Hi Dmitriy Govoruknin, > > > >> > > > > >> > It seems there is a lack of volunteers to apply code inspections > to > > our > > > >> CI > > > >> > & patch submission process. Probably we could come back to your > > > >> > initial idea about setting up inspection locally. > > > >> > > > > >> > Could you commit or share your IDEA inspection settings? I could > > apply > > > >> it > > > >> > at least on my machine and remove odd warning types one by one. > > What do > > > >> you > > > >> > think? > > > >> > > > > >> > Sincerely, > > > >> > Dmitriy Pavlov > > > >> > > > > >> > сб, 4 авг. 2018 г. в 1:22, Dmitriy Pavlov <dpavlov....@gmail.com > >: > > > >> > > > > >> > > Ideally, I should not asking for people to solve something for > me. > > > >> > > > > > >> > > I just want this idea did not disappear from our radar. > > > >> > > > > > >> > > пт, 3 авг. 2018 г. в 23:47, Dmitriy Setrakyan < > > dsetrak...@apache.org > > > >> >: > > > >> > > > > > >> > >> On Fri, Aug 3, 2018 at 7:49 AM, Dmitriy Pavlov < > > > >> dpavlov....@gmail.com> > > > >> > >> wrote: > > > >> > >> > > > >> > >> > > > > >> > >> > I understand it is not so Apache-way from my side to ask > > > >> volunteers to > > > >> > >> do > > > >> > >> > some things (instead of contributing it by myself). > > > >> > >> > > > >> > >> > > > >> > >> Dmitriy, I am not sure why you feel this is not the Apache way. > > No > > > >> one > > > >> > can > > > >> > >> do everything themselves. You should absolutely keep > recruiting > > more > > > >> > >> volunteers from the community. > > > >> > >> > > > >> > >> D. > > > >> > >> > > > >> > > > > > >> > > > > >> -- > > > >> -- > > > >> Maxim Muzafarov > > > >> > > > > > > > -- -- Maxim Muzafarov