Pull Request suggestion or feedback request

49 views
Skip to first unread message

Alan Mehio

unread,
Apr 14, 2021, 6:36:12 AM4/14/21
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
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.
> --
> You received this message because you are subscribed to the Google
> Groups "bitcoinj" group.
> To unsubscribe from this group and stop receiving emails from it, send
> an email to bitcoinj+u...@googlegroups.com
> <mailto:bitcoinj+u...@googlegroups.com>.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/bitcoinj/55052ea9-b970-4e34-b1e9-78bdef4c926bn%40googlegroups.com
> <https://groups.google.com/d/msgid/bitcoinj/55052ea9-b970-4e34-b1e9-78bdef4c926bn%40googlegroups.com?utm_medium=email&utm_source=footer>.

Alan Mehio

unread,
Apr 14, 2021, 9:11:46 AM4/14/21
to bitcoinj
Andreas,
Good to hear your feedback. The SonarLint will indeed refactor specific issue on the entire repository and  yes, it is upp to the reveiwers to  decide. 
Reply all
Reply to author
Forward
0 new messages