Roman Ivanov
Roman Ivanov
@rnveach , please update PR description to not mention old ideas (we can move them to comments to preserve ). Please use PR number in commit, as issue is like...
I recommend to merge, let @rnveach use it in separate project, and then write doc.
@rnveach , please and sub task to big issue and let's move further
@nick-mancuso , please share how filter shuold looks like. I thought I did this yesterday. Need to know where was mistake.
I think first section of VCS rules are outdated. I tried by such rules to reduce execution of TC , if we change unrelated files. But right now TC is...
No filters right now https://teamcity.jetbrains.com/admin/editTriggers.html?id=buildType:Checkstyle_IdeaInspectionsMaster
Thank you very much !! We appreciate your help. it would be awesome is you create UT with list like - https://github.com/checkstyle/checkstyle/pull/6969/files#diff-67a0806621fe751520ab842060ba14fcL106 , see whole change of https://github.com/checkstyle/checkstyle/pull/6969 it show...
It would be good to name all input files for xpath validation with prefix "Input". out ITs inputs are named with such prefix. Even it is meaningful to skip common...
>input file names in the IT tier may conflict with similar files in the Test tier. Package name will be different, no compile issues, ... . What kind of conflict...
>This is why Test inputs are "Input{CheckName}CheckXxxx.java" and IT inputs are "Input{CheckName}Xxxx.java" I do not think it is intentional, it is far from obvious, you have to know project well...