Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add shebang #864

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Add shebang #864

merged 2 commits into from
Oct 31, 2023

Conversation

vanillajonathan
Copy link
Contributor

Add a shebang to the Python scripts.

@walles
Copy link
Contributor

walles commented Oct 12, 2023

If you could chmod a+x them as well that would make them runnable from the command line.

@alecthomas
Copy link
Owner

Agreed, it doesn't really achieve anything without being executable.

@vanillajonathan
Copy link
Contributor Author

But when I run Python scripts, I usually chmod +x first before I run them as ./script.py instead of running python3 script.py. I am sure there are others who do this too.

@walles
Copy link
Contributor

walles commented Oct 14, 2023

You should:

  1. chmod a+x on the scripts
  2. git commit both scripts
  3. git push that change to this branch

git tracks the executable bit, so if you want them executable you should commit after making them executable.

@alecthomas alecthomas merged commit 038fab5 into alecthomas:master Oct 31, 2023
2 checks passed
@vanillajonathan vanillajonathan deleted the patch-2 branch October 31, 2023 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants