treason icon indicating copy to clipboard operation
treason copied to clipboard

Substantial refactor of command logic

Open dnathe4th opened this issue 5 years ago • 3 comments

Resolves #35

dnathe4th avatar Apr 10 '20 01:04 dnathe4th

I like what you've done here. Some of my comments are nit picking, sorry about that. But hey, when you open a pull request just to improve my code, I expect perfection :)

Thanks for taking the time.

octachrome avatar Apr 10 '20 17:04 octachrome

Any progress?

octachrome avatar May 04 '20 10:05 octachrome

Yeah I'm tracking down one bug still, somewhere I'm being inconsistent on where an emission is necessary after a given command. Will try to sort it out there next few days

On Mon, May 4, 2020, 3:27 AM Chris Brown [email protected] wrote:

Any progress?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/octachrome/treason/pull/39#issuecomment-623384258, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACKQ7SX6RTK62PM2PIBCFDRP2KCHANCNFSM4MFET36Q .

dnathe4th avatar May 04 '20 15:05 dnathe4th