abaldwin88 / roamer

The Plain Text File Manager
MIT License
607 stars 15 forks source link

Confirmation (add safety) #28

Closed alex-bender closed 7 years ago

alex-bender commented 7 years ago

This pull request adds a bit of safety to the roamer:

Tested on Debian 8

alex-bender commented 7 years ago

@abaldwin88 Hello! I've created a new PR, please let me know if you want me to make any changes.

alex-bender commented 7 years ago

Please check my last commit

alex-bender commented 7 years ago

If you'd like this commit -- it's needed to add ignore rule to pylint

pylint roamer tests
************* Module roamer.session
W: 17, 4: Redefining built-in 'input' (redefined-builtin)
C: 17, 4: Invalid constant name "input" (invalid-name)
abaldwin88 commented 7 years ago

Yea I'm ok with that. You can disable those checks for that specific line.

alex-bender commented 7 years ago