Skip to content

Fixing squid: S2142 "InterruptedException" should not be ignored#18

Open
devFozgul wants to merge 1 commit intocundong:masterfrom
DevFactory:release/InterruptedException_should_not_be_ignored
Open

Fixing squid: S2142 "InterruptedException" should not be ignored#18
devFozgul wants to merge 1 commit intocundong:masterfrom
DevFactory:release/InterruptedException_should_not_be_ignored

Conversation

@devFozgul
Copy link

This pull request is focused on resolving occurrences of Sonar rule
squid:S2142 - “ "InterruptedException" should not be ignored”.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S2142
Please let me know if you have any questions.
Fevzi Ozgul

@devFozgul
Copy link
Author

Hi Cundong,
When do you think you can check and merge this PR if possible? Your help will be appreciated very much.

@cundong
Copy link
Owner

cundong commented Jun 1, 2016

what are u doing ??

2016-05-23 13:43 GMT+08:00 Fevzi Ozgul notifications@github.com:

Hi Cundong,
When do you think you can check and merge this PR if possible? Your help
will be appreciated very much.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#18 (comment)

沉舟侧畔千帆过,病树前头万木春~

@devFozgul
Copy link
Author

Please read and some other material about it. "Interruptions should not be swallowed." That is why it is signalled by SonarQube analysis as an error to be fixed in the code. Please follow squid: S2142 rule.
http://www.ibm.com/developerworks/library/j-jtp05236/

@devFozgul
Copy link
Author

Hi Cundong, I have not had news from you for a long time. While you are at it, could you please check my PRs and merge if possible? It is very important for my rating.

@devFozgul
Copy link
Author

Hi Cundong, You approved of my contribution to you code after SonarQube quality analysis. So on behalf of my company DevFactory ı am trying to remove Technical Debt from your code. TD is a burden on the software which is checked by a tool like SonarQ. Please take one step further and check my PRs and merge them if there are conflicts. Thank you.

@devFozgul
Copy link
Author

cundong
Hi. This e-mail was returned from you as a reply. I am just reminding you that you gave permission to improve the code quality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants