Skip to content

Remove redundant stdin specifier from cut command in inject_and_init.sh#2718

Merged
plengauer merged 3 commits into
mainfrom
copilot/fix-injection-error
Dec 20, 2025
Merged

Remove redundant stdin specifier from cut command in inject_and_init.sh#2718
plengauer merged 3 commits into
mainfrom
copilot/fix-injection-error

Conversation

Copilot AI commented Dec 20, 2025

Copy link
Copy Markdown
Contributor

CI failure in abom-http repository (run 20011033891, job 58016445475) when using opentelemetry-shell v5.35.0 instrumentation.

Changes

  • Line 87 of actions/instrument/job/inject_and_init.sh: Removed trailing - from cut -d ':' -f 2 -

The - argument explicitly specifies stdin, which is already cut's default behavior. This was the only occurrence of this pattern across the entire codebase - all other cut invocations omit the trailing -.

# Before
cut -d ':' -f 2 - | tr ',' '\n' | ...

# After  
cut -d ':' -f 2 | tr ',' '\n' | ...

This eliminates a code inconsistency in the dependency checking pipeline that could potentially cause edge case failures in certain shell environments.

Original prompt

There is an error here https://github.com/plengauer/abom-http/actions/runs/20011033891/job/58016445475?pr=216 when injecting with this project. find it and fix it!


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: plengauer <100447901+plengauer@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix error in injection process Remove redundant stdin specifier from cut command in inject_and_init.sh Dec 20, 2025
Copilot AI requested a review from plengauer December 20, 2025 10:44
@plengauer plengauer marked this pull request as ready for review December 20, 2025 10:53
Copilot AI review requested due to automatic review settings December 20, 2025 10:53
@plengauer plengauer enabled auto-merge (squash) December 20, 2025 10:53
@plengauer plengauer disabled auto-merge December 20, 2025 10:53
@plengauer plengauer enabled auto-merge (squash) December 20, 2025 10:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Comment thread actions/instrument/job/inject_and_init.sh
@plengauer plengauer merged commit 6a3bb65 into main Dec 20, 2025
1645 of 1655 checks passed
@plengauer plengauer deleted the copilot/fix-injection-error branch December 20, 2025 19:28
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.

4 participants