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

Resolve problems and warnings from static code analysis. #564

Open
LuukBlom opened this issue Oct 29, 2024 · 0 comments
Open

Resolve problems and warnings from static code analysis. #564

LuukBlom opened this issue Oct 29, 2024 · 0 comments
Assignees
Labels
All tabs Issues related to all tabs in FA simultaniously Maintenance Cleanup, admin & other chore to keep the project running Meta issue Epics & tracking issues

Comments

@LuukBlom
Copy link
Contributor

Description

We use Pylance to do some static code analysis, which produces a huge amount of warnings and problems, that we kind of ignore.

Lets at least tackle the easy ones: Add type hints everywhere, checking for None, type mismatches etc.
It should prevent bugs we have not had yet.

Since this is a huge amount of work, I think the bests approach to keep track of it all, is to go through the codebase per feature or even per file.
Then, when you find a feature or file you want to resolve the warnings for:
add a small issue for that feature/file, and add it in the issue list of below.

Issue list

#xxx this is an example feature

Use case

No response

@LuukBlom LuukBlom added Meta issue Epics & tracking issues Maintenance Cleanup, admin & other chore to keep the project running All tabs Issues related to all tabs in FA simultaniously labels Oct 29, 2024
@LuukBlom LuukBlom self-assigned this Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
All tabs Issues related to all tabs in FA simultaniously Maintenance Cleanup, admin & other chore to keep the project running Meta issue Epics & tracking issues
Projects
None yet
Development

No branches or pull requests

1 participant