search

J-Rios / TLG_JoinCaptchaBot

Telegram Bot to verify if users joining a group are human. The Bot sends an image captcha for each new user and kicks any of them who can't solve the captcha in a specified time.
GNU General Public License v3.0
515 stars 217 forks source link

Rename run to run.sh #136

Closed itJunky closed 2 years ago

itJunky commented 2 years ago

Any files need antype for correct interpreting by humans and systems

J-Rios commented 2 years ago

Hello,

What a nice topic to discuss and create a flame war xD

Note that it is just style, there is people that can agree to use .sh extension and others disagree, but not really transcendent, don't care too much for me...

A point to use .sh extension: It allows a human to know directly by the name that it is a shell script.

A point to don't use .sh extension: It increase the file name with a not really pretty extension and also add confusion and complexity to what interpreter should the user use if the user are not familiar with shell scripts (of course the shebang line of the script will do this job for the user, but the user could not be familiar with that).

There is a some recommended style guides by Google, and it said that shell scripts that needs to be executed shouldn't use any extension and just keep the .sh extension for shell script libraries that are not going to be executed directly.

It doesn't care too much for me to change it or not, but just because I'm lazy and going to keep the current style for that script files (note that there is also other shell scripts in the projects, like status and kill).

Thanks, Best Regards.