Closed MattPrit closed 1 year ago
Merging #94 (0ca0d1f) into master (f4b3625) will increase coverage by
0.04%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #94 +/- ##
==========================================
+ Coverage 98.58% 98.62% +0.04%
==========================================
Files 69 69
Lines 2117 2181 +64
==========================================
+ Hits 2087 2151 +64
Misses 30 30
Impacted Files | Coverage Δ | |
---|---|---|
...apters/interpreters/command/command_interpreter.py | 100.00% <100.00%> (ø) |
|
...kit/adapters/interpreters/command/regex_command.py | 100.00% <100.00%> (ø) |
|
tickit/adapters/interpreters/utils.py | 100.00% <0.00%> (ø) |
|
tickit/adapters/interpreters/wrappers/__init__.py | 100.00% <0.00%> (ø) |
|
tickit/__init__.py | ||
...ers/interpreters/wrappers/splitting_interpreter.py | 100.00% <0.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Closing this as not planned. When it becomes desired again raise a new issue, linked to this #177.
Requires #89 Fixes #92 and #93 Required for #90
RegexCommand
to parse from data of typeAnyStr
rather than justbytes
.RegexCommand.parse
now also returns the start and end indices of the match within the message, as well as the length of the message matched against. This gives enough information forMultiCommandInterprerter
to strip off a matched command and forCommandInterpreter
to check for a full match.CommandInterpreter.handle
logic updated to use the new return signature.RegexCommand
andCommandInterpreter