-
Notifications
You must be signed in to change notification settings - Fork 13
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
Chore/clean outdated files #112
base: main
Are you sure you want to change the base?
Conversation
Minimum allowed coverage is Generated by 🐒 cobertura-action against 3c4202d |
@samuel-esp It would be awesome if you could test that i didn't break any functionality. At the moment I cannot do this effectively. I know it's a lot of changes so take your time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mostly LGTM, but we should still test it
Motivation
There were a few outdated files that were pretty much unusable before. This pr aims to clean up a bunch of outdated things and format the code accordingly.
Changes
Tests done
pytest and pre-commit ran with no errors
TODO