search

eclipse / kuksa.val.feeders

kuksa.val.feeders
Apache License 2.0
8 stars 22 forks source link

Improve maintainability and robustness #134

Closed sophokles73 closed 10 months ago

sophokles73 commented 10 months ago

Added typing information to dbcfeeder to make code easier to read and to allow more meaningful linting.

Also refactored some very large functions into multiple smaller functions to improve readability and separation of concern.

erikbosch commented 10 months ago

Looks good to me, any preferences to whether this should be merged before or after the other PR @sophokles73? I have not checked if merge conflicts are expected after the first one has been merged.

sophokles73 commented 10 months ago

this should be merged right away FMPOV