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.

Sign up for GitHub

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

Jump to bottom

Add Unit Tests #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add Unit Tests #6

wants to merge 2 commits into from

Conversation

kalisjoshua
Copy link

No description provided.

@kalisjoshua
Copy link
Author

I don't know if you wanted these but I wanted to tinker with the code and I prefer to have unit tests before doing so.

@kalisjoshua
Copy link
Author

ping

@jonrandy
Copy link
Owner

Sorry, been super busy. TBH I never use automated testing (never have in 38 years of developing). It's highly unlikely ikely that I would maintain them or even run them.

@jonrandy
Copy link
Owner

That said, I'll try and find some time for a proper look, and decide what to do. I think the code has moved on somewhat since you wrote the tests too

@kalisjoshua
Copy link
Author

No worries or pressure. I wanted some tests for myself because I wanted to hack on your solution but wanted to be sure that I wasn't missing any functionality. No rush.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kalisjoshua @jonrandy