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

Hacktober -> more unit tests #39

Closed
sletort opened this issue Sep 27, 2020 · 2 comments
Closed

Hacktober -> more unit tests #39

sletort opened this issue Sep 27, 2020 · 2 comments

Comments

@sletort
Copy link
Contributor

sletort commented Sep 27, 2020

Hello,
Hacktober brings me back to your project. :)

As proposed, in the "Ideas & improvements", I'll make PR to update unit tests and have a better coverage.

@axbazin
Copy link
Member

axbazin commented Sep 27, 2020

Hello,
Thank you in advance then :)
As I recall I think the 'spot' class (which did not exist last october I believe ?) is the only base class missing tests on master atm, but I might be wrong
Any changes will be tested by github actions now, however I don't really know how that goes with PRs

@axbazin axbazin closed this as completed Mar 8, 2021
@axbazin
Copy link
Member

axbazin commented Mar 8, 2021

#41 was the contribution related to this issue.

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

No branches or pull requests

2 participants