-
Notifications
You must be signed in to change notification settings - Fork 40
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
Opportunities for rearchitecture? Refactoring? #358
Comments
I think this is a great idea, but I'm less sure that code and architecture I'd be more inclined to care about usefulness, useability, and actual use. Those are, imo, the places to start any retrospective evaluations of our On Sun, Dec 21, 2014 at 10:15 AM, Phil Wolff [email protected]
Tony Barreca |
I agree, Tony, and that review is being done separately, including being the first app to undergo testing/inquiry with the CUTgroup. You might also want to see some of the wishlist items on our Trello board (https://trello.com/b/KSk0K4fF/opendisclosure-roadmap-brainstorms). We don't have an issue on file to take a breath to clean up code before we dive into the big stuff coming up. I suspect this is a very quick review; the codebase is relatively small. |
Thanks, Phil. It hadn't occurred to me that the CUT group was going to look at already Either way, I am really glad that the group exists! We owe Andrea and Having sat through innumerable code reviews in my life, I certainly think On Sun, Dec 21, 2014 at 11:48 AM, Phil Wolff [email protected]
Tony Barreca |
2015Q1 is likely the best time to review what's been built so far and look for opportunities to make our code and architecture better before building out. Any thoughts on how to do this? Group code walkthroughs?
The text was updated successfully, but these errors were encountered: