AAClause / BrailleExtender

NVDA add-on that improves braille support
https://andreabc.net/projects/NVDA_addons/BrailleExtender/
GNU General Public License v2.0
16 stars 17 forks source link

Code cleanup: use Flake8 linter #68

Open josephsl opened 4 years ago

josephsl commented 4 years ago

Hi,

Stemming from #31 discussion:

As a first step in preparing portions of Braille Extender to become NVDA Core pull requests, use Flake8 linter tool to clean up code. Although the Flake8 configuration to be used is not really the same as the one employed by NVDA Core, it has similar ideas, namely keywords, import statements, spacing and such.

Implementation:

  1. Clone Braille Extender source code.
  2. Run Flake8 against the source code. Ignore bits about line length, tqbs and such.
  3. Correct issues noted, run Flake8, and test the changes in real life with braille displays.

Note: if the accompanying pull request is integrated, you don't have to mention Flake8.

Thanks.