add comment about naive regular expression.

Opentrons · Mar 17, 2021 · 72148b2 · 72148b2
1 parent 9a5eb16
commit 72148b2
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/api/src/opentrons/hardware_control/emulation/app.py b/api/src/opentrons/hardware_control/emulation/app.py
@@ -19,7 +19,13 @@
 
 
 LINE_REGEX = re.compile(r"(\S+)(.+)")
-"""Split the line into command and payload"""
+"""Split the line into command and payload.
+
+TODO AL 20210222 This regex is a very naive approach and should be revisited
+ if we are going to expand emulator support.
+We can use a regex like "[A-Z][0-9]+\\.*[0-9]*" to match gcodes (i think). And
+handle our custom directives like 'version' and 'dfu' separately.
+"""
 
 
 class ConnectionHandler: