idank / bashlex

Python parser for bash
GNU General Public License v3.0
550 stars 94 forks source link

Added support for alphanumeric variable names (Fixes #66) #67

Closed BlankCanvasStudio closed 2 years ago

BlankCanvasStudio commented 2 years ago

This change enforces that the first char only be a character or a '_'. After that, any alphanumeric char is acceptable. This is in line with bash's naming convention. It passes all the tests

idank commented 2 years ago

Can you please add tests?

BlankCanvasStudio commented 2 years ago

Added testing.

idank commented 2 years ago

This looks great, can you please squash the commits and change the summary line of the commit message to follow the existing convention? Thanks!