-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat add uber fx #331
Feat add uber fx #331
Conversation
Codecov Report
@@ Coverage Diff @@
## master #331 +/- ##
==========================================
+ Coverage 14.19% 14.20% +0.01%
==========================================
Files 31 31
Lines 2558 2556 -2
==========================================
Hits 363 363
+ Misses 2166 2164 -2
Partials 29 29
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…ssion to bind to 0.0.0.0
Tested locally, works good. Found a few bugs in UI. |
Are the UI bugs documented? Anyway, looks very good to me, very good job with breaking down of master.go, and I like how fx does dependency injection |
No, not documented. Just manually breaking the ui and fixing it in a branch. I like uber fx approach to dependency injection as well! |
Replace https://github.com/uber-go/dig with https://github.com/uber-go/fx to string the components together.
Add Procfile to run cockroach, nsq, and server on local machine easier with goreman or any alternative (https://github.com/ddollar/foreman#ports)