Roman Ivanov
Roman Ivanov
Please fix codenarc meanwhile
@relentless-pursuit , please reply done to review items, if you are ready for review
@relentless-pursuit , please add to your PR doc update https://github.com/checkstyle/contribution/commit/f10f82a9c036fc4565b4868d7cc8b0a3fe6eea2d#diff-1d82d916db8f81ee95ff589f11bde9990f0b83f950957a48e4a573d77e6fff5fR75
Testing should be there https://github.com/checkstyle/checkstyle/pull/14662
Please add to https://github.com/checkstyle/contribution/blob/3decd5c38c3e225b9fd00f299199843b25aeb8b0/appveyor.yml#L85
failure was: ``` Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found." At line:3 char:3 + (new-object System.Net.WebClient).DownloadFile( + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + CategoryInfo : NotSpecified:...
@rnveach , are you aware about this PR ? it is related to your PR https://github.com/checkstyle/contribution/pull/747
In property file , first value of line is name , we could define any name there and use the same repo few times
all we need is just switch to "master", if it works - lets keep it.
I do not remember this discussion, I forgot a reason to keep old(versioned config line).