search

Nextomics / nextsv

MIT License
33 stars 8 forks source link

language suffix on user-facing program: nextsv.pl vs nextsv #2

Closed 0xaf1f closed 7 years ago

0xaf1f commented 7 years ago

For this user-facing script, it's generally better not to expose implementation details like the language used to write the program. See https://wiki.debian.org/UpstreamGuide#Language_extensions_in_scripts for more details.

You already have nextsv.pl marked executable and a proper shebang line, so there is no need to keep the .pl suffix. In line with this, the documentation should probably be updated to instruct usage as ./nextsv or to put the folder on the PATH and call it as simply nextsv.

fangli80 commented 7 years ago

Thank you for your suggestion. We will modify this in the next release.