Manish Krishna Kandrakota

Results 86 comments of Manish Krishna Kandrakota

@rnveach , @romani I was exploring the Issue page while this issue came into limelight.... the issue seems to be open for almost 4-5 years with no merges. Is this...

> Where does the first post not share these details? @rnveach , The description explains the requirements precisely. :) I just wanted to know if there were any newer updates...

@rnveach , @romani I need clarification regarding discussions from https://github.com/checkstyle/checkstyle/issues/6207#issuecomment-523013433 and https://github.com/checkstyle/checkstyle/issues/6207#issuecomment-522679928 > What if we name it as InputXpathAbstractClassNameInner.java ? to put Xpath as distinguishing marker from inputs of...

Also, Once I am done with the naming convention , I will organize our It codebase folder-wise matching out test folders. Doing this parallely would be a bit confusing and...

> I have attached a PR #14595 , @rnveach , @romani In the PR, I have added another naming pattern for the test to validate. So that both the old...

> We need all sequence of PRs to close this issue. > @MANISH-K-07 , please split your big PR in multiple and send them to us. > Single review only...

> I can understand if you feel you have higher priority issues. Actually yes @rnveach , I'm working on an indentation issue and its taking up some time for investigation...

**`// Outdated Comment`** **`// Refer PR description for latest`** > Please investigate https://circleci.com/gh/checkstyle/checkstyle/494961 @nrmancuso , I have found the root cause of the failing CI and am working on implementing...

**`// Outdated Comment`** **`// Refer PR description for latest`** @romani , @nrmancuso Please excuse me for the multiple pushes but I have been trying to resolve the issue with [ci/circleci:...

**`// Outdated Comment`** **`// Refer PR description for latest`** @romani , @nrmancuso **_Please Review._** > We need to understand why before we start trying to change versions in a bunch...