-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Add a "close" method to the API #39
Conversation
Wouldn't it make more sense to define It's a much smaller change then and the restructuring wouldn't be needed. |
That does make more sense. Thanks. I'll fix it. |
Maybe a rebase would be cleaner? (Still learning. Sorry.) |
Hm, probably. Especially since |
I suck :D Almost there :) |
So simple, huh? :) Thanks for your patience. |
Also export it and I'll merge ;) Edit: Oh wait, is that even necessary?! It isn't, right? If so, nevermind. |
Add a "close" method to the API
Note that since
depends on @ dot, I moved api.jl to AtomShell so @ dot is defined first.
Closes #28