-
Notifications
You must be signed in to change notification settings - Fork 24
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
Audit #14
Comments
I agree! I already heard comments from people that we should refactor a few things. I hope to write it up somewhere between today and tomorrow. |
Great. Good luck. NK On Mon, Dec 3, 2012 at 1:19 PM, DrWhax notifications@github.com wrote:
|
Agree! And this is the primary reason why it's not tag "stable". Peer review is critical for this kind of project so any comment is a good comment (well if not done on twitter as a complain :). Thanks Nadim! |
Let's see if we could find some people who are interested in reviewing the code. |
We learned the hard way over at Cryptocat to have our shit audited properly
The text was updated successfully, but these errors were encountered: