Closed Bug 1937899 Opened 2 months ago Closed 1 month ago

[wpt-sync] Sync PR 49739 - Clean-up argparse usage

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox136 fixed)

Status:
RESOLVED FIXED
Milestone:
136 Branch
Tracking Flags:
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 49739 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/49739
Details from upstream follow.

Sam Sneddon <gsnedders@apple.com> wrote:

Clean-up argparse usage

We frequently pass action="store" (the default), set dest equal to the default (computed from the option name), and set default equal to the default. All of these are unnecessary.

This does some light reformatting, partly based on autopep8 with the expected ignores.

@jonathan-j-lee mentioned this as a nit in https://github.com/web-platform-tests/wpt/pull/49630#discussion_r1879094353, so, uh, let's fix all of these.

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9f6dae846a16 [wpt PR 49739] - Clean-up argparse usage, a=testonly
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/9f6dae846a16

Status: NEW → RESOLVED
Closed: 1 month ago
status-firefox136: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.