search

m0bilesecurity / RMS-Runtime-Mobile-Security

Runtime Mobile Security (RMS) 📱🔥 - is a powerful web interface that helps you to manipulate Android and iOS Apps at Runtime
https://twitter.com/mobilesecurity_
GNU General Public License v3.0
2.62k stars 376 forks source link

Code cleanup: Remove unused imports, mark unused vars & simplify ... #25

Closed ghost closed 4 years ago

ghost commented 4 years ago

... if-condition.

This commit cleans up the code and improves readability and consistency by removing unused imports, clearly marking unused vars by prefixing them with an underscore and simplifying the if condition.

Find the other issues found here - https://deepsource.io/gh/chsatyap/RMS-Runtime-Mobile-Security/issues/?category=all

This PR also adds .deepsource.toml configuration file to run DeepSource analysis on the repo with.

Upon enabling DeepSource, the analysis will run on every PR and commit to detect 560+ types of issues in the changes — including bug risks, anti-patterns, security vulnerabilities, etc.

To enable DeepSource analysis after merging this PR, please follow these steps:

  1. Signup on DeepSource with your GitHub account and grant access to this repo.
  2. Activate analysis on this repo here. You can also look at the docs for more details.

I work for DeepSource. Please do let me know if I can be of any help!

m0bilesecurity commented 4 years ago

hey @chsatyap, many thanks for the PR. Very useful 🚀 Is it possible to create and activate my own repo on DeepSource? Best

ghost commented 4 years ago

Absolutely! You can active DeepSource on any repo as long as you sign up on the site with your GitHub. Lmk if there's anything else you need!