You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to bitcoinj
Dear all,
It would be nice to hear from you first before I start to make a pull request which will try to fix the java static analysis issues based on SonarLint.
Is there any wish list to todo or SonarLint is OK to start with first ?
I would like to receive a feedback from the community
Regards,
Alan
Andreas Schildbach
unread,
Apr 14, 2021, 6:41:03 AM4/14/21
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to bitc...@googlegroups.com
It depends on the issue being fixed. Not all Lint warnings are properly
fixable, and some are not even desirable to fix (in my eyes).
For this kind of PR, I prefer PRs that refactor a specific issue on the
entire repository. Then, the discussion on wether it's an improvement or
not can happen in the comments on the PR.