Issue 11 change pylint on commit

[HelloFive]

Befor putting PR (put x in bracket if you ran it.);

• [x] Run pylint $(git ls-files '*.py') passes

Overview

Describe your pull request. Attach github issue link

• Git Issue: https://github.com/dhmoon91/porobot/issues/11

Changes:

• Modify pre-push to pre-commit
• Add on README with steps to disable pre-push and enable pre-commit
• Resolve pylint command mismatches (detailed information is available --> https://github.com/dhmoon91/porobot/issues/11#issuecomment-892391634) (from https://github.com/dhmoon91/porobot/runs/3238335046?check_suite_focus=true , we can see that the pylint result is identical with local run screenshot posted above)
• Resolve existing pylint issues
• Updated PR template with new pylint command pylint --rcfile=.pylintrc $(git ls-files '*.py') which will run against all *.py in this directory

[dhmoon91]

nice - Could you remove disabling comments that were added to .pylintrc?

eg; # pylint: disable=too-many-locals, too-many-branches, too-many-statements remove theses lines and I thinkg we can ignore broad-except on error catch as well