Skip to content

Conversation

@tleonhardt
Copy link
Member

@tleonhardt tleonhardt commented Jul 13, 2017

Bugs fixed:

  • Case-sensitive parsing was completely broken, this has been fixed
  • +D to quit wasn't working when case-sensitive parsing was enabled, this is fixed

Also added a couple unit tests.

Added an example to the examples directory.

This closes #188

Bugs fixed:
- Case-sensitive parsing was completely broken, this has been fixed
- <Ctrl>+D to quit wasn't working when case-sensitive parsing was enabled, this is fixed

Added a "case_sensitive.py" example in the examples directory for quickly testing case-sensitive command parsing behavior.
Also:
- Added a couple unit tests related to case-sensitive vs case-insensitive parsing
- Updated CHANGES.md
@tleonhardt tleonhardt added this to the 0.7.6 milestone Jul 13, 2017
@tleonhardt tleonhardt self-assigned this Jul 13, 2017
@codecov
Copy link

codecov bot commented Jul 13, 2017

Codecov Report

Merging #189 into master will increase coverage by <.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
+ Coverage   97.07%   97.08%   +<.01%     
==========================================
  Files           1        1              
  Lines        1129     1131       +2     
==========================================
+ Hits         1096     1098       +2     
  Misses         33       33
Impacted Files Coverage Δ
cmd2.py 97.08% <75%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f517ff8...b6bf292. Read the comment docs.

@tleonhardt tleonhardt merged commit 30a3b66 into master Jul 13, 2017
@tleonhardt tleonhardt deleted the case_bugs branch July 13, 2017 00:43
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.

Case-sensitive command parsing is broken

2 participants