Skip to content

Update draft changelog script shebang#2450

Merged
breedx-splk merged 4 commits intoopen-telemetry:mainfrom
breedx-splk:update_shebang
Nov 20, 2025
Merged

Update draft changelog script shebang#2450
breedx-splk merged 4 commits intoopen-telemetry:mainfrom
breedx-splk:update_shebang

Conversation

@breedx-splk
Copy link
Copy Markdown
Contributor

This script for drafting changelog entries uses -A which is unknown on the old/legacy version of bash that ships with macos....which for many of us means then that the script is unusable.

The workaround, like so many things, is to use a version of bash from homebrew...but when you do that the script still won't use it unless the shebang is updated to use env.

@breedx-splk breedx-splk requested a review from a team as a code owner November 20, 2025 19:08
Copy link
Copy Markdown
Member

@trask trask left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script for drafting changelog entries uses -A which is unknown on the old/legacy version of bash that ships with macos....which for many of us means then that the script is unusable.

I'd also be fine if there's a different way to achieve the same result on older versions of bash

@breedx-splk
Copy link
Copy Markdown
Contributor Author

This script for drafting changelog entries uses -A which is unknown on the old/legacy version of bash that ships with macos....which for many of us means then that the script is unusable.

I'd also be fine if there's a different way to achieve the same result on older versions of bash

I don't know of one. I think it's fine the way it is...and I think that using env for the shebang is preferred/best practice anyway?

@breedx-splk breedx-splk merged commit 48b491d into open-telemetry:main Nov 20, 2025
22 checks passed
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