Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Enhancement] HttpUtils PMD check warning #2633

Closed
2 tasks done
jonyangx opened this issue Dec 21, 2022 · 0 comments · Fixed by #2634
Closed
2 tasks done

[Enhancement] HttpUtils PMD check warning #2633

jonyangx opened this issue Dec 21, 2022 · 0 comments · Fixed by #2634
Assignees
@jonyangx
Labels
enhancement New feature or request

Comments

@jonyangx
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar issues.

Enhancement Request

1.AvoidReassigningParameters
2.Unusedimports
3.compareObjectsWithEquas.

Describe the solution you'd like

refactor code
1.new local variable.
2.delte unused imports
3.repalce == with equals method.

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@jonyangx jonyangx added the enhancement New feature or request label Dec 21, 2022
@jonyangx jonyangx self-assigned this Dec 21, 2022
@jonyangx jonyangx mentioned this issue Dec 21, 2022
Merged
xwm1992 added a commit that referenced this issue Dec 26, 2022
@xwm1992
Merge pull request #2634 from jonyangx/issue2633
0d26e39 
[ISSUE #2633] HttpUtils PMD check warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonyangx jonyangx

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #2633] HttpUtils PMD check warning
1 participant
@jonyangx