Skip to content
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

introduce sub commands #209

Merged
merged 12 commits into from
Dec 22, 2019
Merged

introduce sub commands #209

merged 12 commits into from
Dec 22, 2019

Conversation

ktr0731
Copy link
Owner

@ktr0731 ktr0731 commented Dec 21, 2019

Based on #190, we introduced a sub-command mechanism into Evans.
Hereby, the previous command-line interface will be deprecated in the future by releasing v1.0.0.
This implementation doesn't break the previous interface from the aspect of backward-compatibility. So, we can also use the previous one, but it displays a warning message when finishing Evans.

Also, by introducing this change, it is possible to implement CLI inspecting commands such that #189.

See #190 for example usage.

@ktr0731 ktr0731 self-assigned this Dec 21, 2019
@codecov
Copy link

codecov bot commented Dec 22, 2019

Codecov Report

Merging #209 into master will decrease coverage by 0.15%.
The diff coverage is 81.13%.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   80.65%   80.49%   -0.16%     
==========================================
  Files          44       45       +1     
  Lines        2305     2389      +84     
==========================================
+ Hits         1859     1923      +64     
- Misses        280      291      +11     
- Partials      166      175       +9

@ktr0731 ktr0731 marked this pull request as ready for review December 22, 2019 09:59
@ktr0731 ktr0731 merged commit 33d0d09 into master Dec 22, 2019
@ktr0731 ktr0731 deleted the sub-commands branch December 22, 2019 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant