Skip to content
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

First linting pass #92

Merged
merged 1 commit into from
Nov 29, 2021
Merged

First linting pass #92

merged 1 commit into from
Nov 29, 2021

Conversation

Avasam
Copy link
Collaborator

@Avasam Avasam commented Nov 16, 2021

This shows all the changes from the first linting passe. Not everything has been tested.

  • Configured VSCode and Markdownlint settings
  • Added and configured the following linters as strict as I could: pylint, flake8, mypy, bandit, pyright
  • Partially filled in some type stubs: cv2, imagehash, keyboard, pyautogui, pythonwin, win32helper
  • [Breaking change] Using interpolation=cv2.INTER_NEAREST everywhere as it is the fastest interlpolation method
  • Hide generated/compiled files
  • SelectRegionWidget width and height as functions
  • Added validate_images_before_parsing() to validate images both before maxFPS and starting autosplitter

The following is still to be done and will come in future PRs:

  • Configure and fix casing
  • AutoSplit x, y, height, width attributes override PyQt window functions
  • Black formatting
  • Fix all errors and most warnings left
  • Continuous Integration

External linked issues:
kaluluosi/pywin32-stubs#5
kaluluosi/pywin32-stubs#6
kaluluosi/pywin32-stubs#7
pyinstaller/pyinstaller-hooks-contrib#110
JohannesBuchner/imagehash#151
boppreh/keyboard#505
asweigart/pyautogui#645
microsoft/pylance-release#2089
numpy/numpy#20472

@Avasam Avasam marked this pull request as draft November 16, 2021 21:43
@Avasam Avasam changed the base branch from avasam-complete to master November 27, 2021 23:48
@Avasam Avasam changed the base branch from master to dev November 28, 2021 02:54
@Avasam Avasam changed the base branch from dev to 2.0.0 November 29, 2021 13:16
@Avasam Avasam marked this pull request as ready for review November 29, 2021 13:18
@Avasam Avasam marked this pull request as draft November 29, 2021 13:54
Configured VSCode and Markdownlint settings
Added and configured the following linters as strict as I could: pylint, flake8, mypy, bandit, pyright
Partially filled in some type stubs: cv2, imagehash, keyboard, pyautogui, pythonwin, win32helper
[Breaking change] Using interpolation=cv2.INTER_NEAREST everywhere as it is the fastest interlpolation method
Hide generated/compiled files
SelectRegionWidget width and height as functions
Added validate_images_before_parsing() to validate images both before maxFPS and starting autosplitter
@Avasam Avasam marked this pull request as ready for review November 29, 2021 14:22
@Avasam Avasam changed the title Linting First linting pass Nov 29, 2021
@Toufool Toufool merged commit 667696c into Toufool:2.0.0 Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants