Skip to content

Remove shebangs from python files since theyre compiled to pyz anyway#76

Merged
gpalmz merged 1 commit into
mainfrom
gpalmz/azure-remove-shebang
Nov 4, 2025
Merged

Remove shebangs from python files since theyre compiled to pyz anyway#76
gpalmz merged 1 commit into
mainfrom
gpalmz/azure-remove-shebang

Conversation

@gpalmz
Copy link
Copy Markdown
Collaborator

@gpalmz gpalmz commented Nov 4, 2025

Script is still interpreted as python when run as seen in quickstart:

grant.palmer@COMP-JF7QV5QJJH azure % python ./integration_quickstart/dist/azure_integration_quickstart.pyz
Missing required environment variables: DD_APP_KEY, DD_API_KEY, DD_SITE, WORKFLOW_ID

The shebangs also aren't needed at all if we're already running the script with the python command directly

@gpalmz gpalmz requested a review from benjjs November 4, 2025 16:06
@gpalmz gpalmz requested a review from a team as a code owner November 4, 2025 16:06
@gpalmz gpalmz merged commit e2135fd into main Nov 4, 2025
2 checks passed
@gpalmz gpalmz deleted the gpalmz/azure-remove-shebang branch November 4, 2025 16:07
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.

2 participants