Nick Mancuso
Nick Mancuso
>Me & @nrmancuso thinks that in records compact constructor should be first in the order. I didn't think this, I just shared a possibility :) >Initially this module was inspired...
Please investigate https://circleci.com/gh/checkstyle/checkstyle/494961
@MANISH-K-07 please put this PR into draft mode while you work on resolving failure in release-dry-run, then take it out of draft when ready for review
@MANISH-K-07 > Removing the parent POM is somehow initiating the error with this plugin. We need to understand why before we start trying to change versions in a bunch of...
@MANISH-K-07 please do: 1. Link to parent pom config in PR description 2. Aggregate all **relevant** details from various comments into PR description 3. Mark outdated comments in PR as...
@MANISH-K-07 effective PR descriptions, issue descriptions, and comments should be very concise and easy to read. I would expect the PR description to look like: ------------- In order to remove...
@MANISH-K-07 Please hide all outdated comments as requested in https://github.com/checkstyle/checkstyle/pull/14397#issuecomment-1924041690 now that we have a good summary in the PR description
Rebased on latest master
@mahfouz72 when we move onto bug fixes, new features, etc. like this PR (awesome!), please always be the first reviewer. Imagine you have never seen this code before, and call...
@relentless-pursuit ping, do you plan to continue this PR?