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

[Update] add more tests #111

Open
4 tasks
yowainwright opened this issue Apr 5, 2018 · 2 comments
Open
4 tasks

[Update] add more tests #111

yowainwright opened this issue Apr 5, 2018 · 2 comments

Comments

@yowainwright
Copy link
Contributor

Add more tests

For this issue more testing will be implemented.

Reasons 🤔

  • easier code review
  • tests need to be added for future features like:
    • post-merge publishing and changelog updates,
    • automerge

Steps 📊

The checklist key items will be in separate PRs for readability.
Versioning for these PRs will try to best support the community.


Key Items ✅

@yowainwright
Copy link
Contributor Author

@rwwagner90 will manage open issues. I am closing this pr. This issue will not be done.

@RobbieTheWagner
Copy link
Contributor

Adding more tests is probably a good idea. We can keep this open.

@yowainwright yowainwright reopened this Apr 12, 2018
@yowainwright yowainwright removed their assignment May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants