search

knrafto / language-bash

Parse and pretty-print Bash shell scripts
BSD 3-Clause "New" or "Revised" License
35 stars 9 forks source link

Fixes #27 #30

Closed mmhat closed 4 years ago

mmhat commented 4 years ago

Please do not merge until we settle for an appropriate pretty printer output (#24).

mmhat commented 4 years ago

@knrafto tests/pretty/heredoc-nested.sh cannot be parsed but 'bash -n' says it's ok. Can you have a look?

knrafto commented 4 years ago

This looks fantastic, thank you so much! Thanks for finding the bug too, I pushed a fix to master (ab810a7).

mmhat commented 4 years ago

Ok, this one is ready to be merged.

knrafto commented 4 years ago

Sorry for the flaky test, but this one is a build error. Maybe use filter (\fn -> takeExtension fn == ".sh") instead

mmhat commented 4 years ago

Ok, done.