Fixes/2017/ignore serial version UI d

[atulgpt]

Description

Filter the serialversionUID from the analysis
Fixes #2017

Checklist

• PR description added
• tests are added
• KtLint has been applied on source code itself and violations are fixed
• documentation is updated
• CHANGELOG.md is updated

[paul-dingemans]

This PR contains commits abfbbfa and e6ca5a7 which do not belong to this PR. Please ensure that a PR only contains changes relevant to that PR.

[paul-dingemans]

I have update your PR.

Next time, please update changelog.

Also, in chained methods please write:

identifier
    .foo {
       
    }?.bar {

   }

instead of

identifier
    .foo {
       
    } // ==> should be merged with next line         
    ?.bar {

   }

[paul-dingemans]

I can (will) not merge this PR. It still contains changes of another PR which results in merge conflicts above. Please start a new PR and only commit the relevant changes to that PR. Please whenever you start a new PR, please be sure to create a new branch from latest remote master.

[atulgpt]

I can (will) not merge this PR. It still contains changes of another PR which results in merge conflicts above. Please start a new PR and only commit the relevant changes to that PR. Please whenever you start a new PR, please be sure to create a new branch from the latest remote master.

Hi, @paul-dingemans sorry for the inconvenience. let me look into it. If you see the file changes you will see that there is no changes from the previous PR. And if I remember correctly I dropped those commits hard pushed yesterday. But anyway let me look into it again

[atulgpt]

identifier
.foo {

} // ==> should be merged with next line         
?.bar {

}

Hi @paul-dingemans just to be clear you are suggesting to use below style?

identifier
    .foo {
       
    }?.bar {

   }

I am asking as the existing code in PropertyNamingRule uses below style

node
    .takeIf { node.elementType == PROPERTY }
    ?.let { property -> visitProperty(property, emit) }

[paul-dingemans]

No, I meant that

}
?.

Should be on same line, like

}?.