-
Notifications
You must be signed in to change notification settings - Fork 55
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.
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
fix: rename variables as per Codacy review #338
fix: rename variables as per Codacy review #338
Conversation
👇 Click on the image for a new way to code review
Legend |
d19d9f6
to
025ec32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@A-5ingh, the changes look good and I've verified they are reported as fixed on Codacy.
Thanks for the contribution! :)
@all-contributors please add @A-5ingh for code |
I've put up a pull request to add @A-5ingh! 🎉 |
No worries! If there are other issues which are not taken up by anyone I can contribute there as well. |
@A-5ingh now that hacktoberfest is slowing down with less contributors signing up for issues, you can take #332 as well if you'd like :) |
Sure, please assign #332 to me as well. |
@jmakhack can you please add the label |
Associated Issue
Closes #333
Implemented Solution
Corrected the variable names and occurrences as per Pascal casing.
Testing Details
Operating System: MacOS 12.6
RuneLite Version: 1.1.1
Screenshots