-
Notifications
You must be signed in to change notification settings - Fork 0
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
Bug squashing #14
Bug squashing #14
Conversation
Not perfect maybe with every feature but I think the basic functionality is there
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.
Also looks like some NRT warnings that would be worth addressing
Co-authored-by: Kevin B <Keboo@users.noreply.github.com>
5c1ed88
to
598965a
Compare
598965a
to
2d66f20
Compare
Ignoring tests that I broke in the process of fixing the mainly used method, so need to go back later and clean up the tests and other options, but want to get a working version to nuget for mark to use in the meantime. |
Description
Try to fix the bugs in this. Specifically the updating of listings and tests to be correct.