-
Notifications
You must be signed in to change notification settings - Fork 1
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
Config release docs #44
Conversation
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.
Nicely done! I guess there's not much time so it was a pragmatic choice but felt a bit light on testing
class DiffElement: | ||
"""A potential update to the local store.""" |
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.
ooh fun
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.
Yeah, I wanted to be able to separate out the gathering of the diffs from the acceptance of them, so that required this class.
281707e
to
a98a2bc
Compare
Lots of changes to make qw demoable.
Closes #16, closes #25, closes #39
Addresses (no tests) #17, #36