-
Notifications
You must be signed in to change notification settings - Fork 149
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
Use ~/.cql/ directory as default config location. #233
Conversation
laodouya
commented
Jan 24, 2019
•
edited
edited
- Change all cmd config param to ~/.cql/config.yaml
- Config generate will default use ~/.cql/ as well.
- Driver can reduce client.Init func call if expected using ~/.cql/
- Update all unit test.
- Update all README.md
- Update client example.
- Compatible with elder usage.
- Skip compatibility test if branch name contains '/beta_'
Codecov Report
@@ Coverage Diff @@
## develop #233 +/- ##
===========================================
- Coverage 77.85% 77.57% -0.28%
===========================================
Files 150 150
Lines 11914 11931 +17
===========================================
- Hits 9276 9256 -20
- Misses 1946 1977 +31
- Partials 692 698 +6 |
Codecov Report
@@ Coverage Diff @@
## develop #233 +/- ##
==========================================
- Coverage 77.85% 77.6% -0.26%
==========================================
Files 150 150
Lines 11914 11931 +17
==========================================
- Hits 9276 9259 -17
- Misses 1946 1974 +28
- Partials 692 698 +6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.