Skip to content
This repository has been archived by the owner on Jan 30, 2018. It is now read-only.

updated a lot - I think it still all works #57

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

Conversation

tburgin
Copy link

@tburgin tburgin commented Jun 20, 2017

@MagerValp
Copy link
Owner

Thank you, the codebase is in dire need of a clean up. However, I'm afraid I can't merge this. The new indentation adds a lot of noise, but even if I filter that out the changes are too many for a single commit. Feel free to fork the project and release off of your branch though, I'm looking for a new maintainer and this is a perfect start.

@tburgin
Copy link
Author

tburgin commented Jun 21, 2017

There is a lot of noise, sorry about that 😜. Once I started I couldn't stop!
Unfortunately I don't have the bandwidth to be a maintainer. It is a really neat use of a document based app though, I learned a lot about them by digging into this.

@tburgin tburgin closed this Jun 21, 2017
@MagerValp
Copy link
Owner

If you don't mind I'll leave the PR open, in case someone else wants to step in.

@tburgin
Copy link
Author

tburgin commented Jun 21, 2017

No problem. Just want to make sure this isn't a blocker for greg's update to the shadow hash data.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants