Skip to content

[test] Remove decode_output argument to shared.run_process. NFC#8134

Open
sbc100 wants to merge 1 commit into
mainfrom
decode_output
Open

[test] Remove decode_output argument to shared.run_process. NFC#8134
sbc100 wants to merge 1 commit into
mainfrom
decode_output

Conversation

@sbc100

@sbc100 sbc100 commented Dec 16, 2025

Copy link
Copy Markdown
Member

There was only a single user of this argument in the codebase.

@sbc100 sbc100 requested review from kripken and tlively December 16, 2025 00:04
Comment thread test/unit/test_features.py
Comment thread scripts/test/shared.py Outdated
@sbc100 sbc100 enabled auto-merge (squash) December 16, 2025 00:30
@sbc100 sbc100 disabled auto-merge December 16, 2025 00:31
@sbc100 sbc100 enabled auto-merge (squash) December 16, 2025 00:31
@sbc100 sbc100 force-pushed the decode_output branch 3 times, most recently from 20d5127 to 5ffa37c Compare December 18, 2025 16:55
@sbc100 sbc100 requested a review from a team as a code owner March 23, 2026 22:48
@sbc100 sbc100 changed the title [test] Remove decode_output argument to shared.run_process. NFC [test] Remove decode_output argument to shared.run_process. NFC Mar 23, 2026
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