Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement unit testing and fix bug in add.go #21

Merged
merged 14 commits into from
Jul 14, 2022
Merged

Implement unit testing and fix bug in add.go #21

merged 14 commits into from
Jul 14, 2022

Conversation

tbhaxor
Copy link
Contributor

@tbhaxor tbhaxor commented Jul 12, 2022

Fixes #17

@tbhaxor
Copy link
Contributor Author

tbhaxor commented Jul 13, 2022

@Aarush-Goyal This is why (143d6ae) unit tests are required in the projects 😄

@tbhaxor tbhaxor marked this pull request as ready for review July 13, 2022 20:32
@tbhaxor
Copy link
Contributor Author

tbhaxor commented Jul 13, 2022

@Aarush-Goyal Feel free to test this on your system and merge it.

Copy link
Member

@Aarush-Goyal Aarush-Goyal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Aarush-Goyal Aarush-Goyal merged commit aef338c into automation-co:main Jul 14, 2022
@tbhaxor tbhaxor deleted the feat/unittesting branch July 14, 2022 04:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT] Implement unit tests
2 participants