sirix Fix Sonar issues

Fix Sonar issues

Open JohannesLichtenberger opened this issue 3 years ago • 4 comments

https://sonarcloud.io/summary/overall?id=sirixdb_sirix

Oct 25 '21 19:10 JohannesLichtenberger

@JohannesLichtenberger I am working on this, Please assign this to me.

Oct 26 '21 07:10 AnishaSrivastava1411

BTW: I think some of the issues are not real issues, regarding the long / Long autoboxing for instance, but even there I might be wrong.

Oct 26 '21 07:10 JohannesLichtenberger

Hello @JohannesLichtenberger Can I work on this? I am available to fix a few sonar issues. thanks.

Dec 12 '21 07:12 gvdutra

Of course :-)

Dec 12 '21 09:12 JohannesLichtenberger

Hi @JohannesLichtenberger Raised a PR: https://github.com/sirixdb/sirix/pull/545

Oct 31 '22 12:10 slashgk

Hi @JohannesLichtenberger Raised a PR: https://github.com/sirixdb/sirix/pull/667

Oct 04 '23 07:10 slashgk

Hi @JohannesLichtenberger Raised a PR: https://github.com/sirixdb/sirix/pull/669

Oct 04 '23 17:10 persy97

@persy97 that's not enough for a meaningful contribution, sorry

Oct 04 '23 17:10 JohannesLichtenberger

@JohannesLichtenberger Pushed some more sonar fixes in the same PR. Please take a look.

Oct 07 '23 06:10 persy97

Thanks, but I think a cast is missing :)

Oct 07 '23 11:10 JohannesLichtenberger

Can you trigger the build again?

Oct 07 '23 11:10 persy97

Hi @JohannesLichtenberger raised a PR: #671 can someone from maintainer approve?

Oct 14 '23 05:10 Naman0133

Hi @JohannesLichtenberger raised a PR: https://github.com/sirixdb/sirix/pull/674 can someone from maintainer approve?

Oct 26 '23 10:10 Salonimalhotra

Hi @JohannesLichtenberger raised a PR: https://github.com/sirixdb/sirix/pull/676 can someone from maintainer approve?

Oct 27 '23 04:10 smirti-barnwal