Bump yapf from 0.29.0 to 0.43.0 in /sdks/python

[dependabot]

Bumps yapf from 0.29.0 to 0.43.0.

Changelog

Sourced from yapf's changelog.

Change Log

All notable changes to this project will be documented in this file.

This project adheres to Semantic Versioning.

(0.41.0) UNRELEASED

Added

• New DISABLE_SPLIT_LIST_WITH_COMMENT flag. DISABLE_SPLIT_LIST_WITH_COMMENT is a new knob that changes the behavior of splitting a list when a comment is present inside the list.

  Before, we split a list containing a comment just like we split a list containing a trailing comma: Each element goes on its own line (unless DISABLE_ENDING_COMMA_HEURISTIC is true).

  This new flag allows you to control the behavior of a list with a comment separately from the behavior when the list contains a trailing comma.

  This mirrors the behavior of clang-format, and is useful for e.g. forming "logical groups" of elements in a list.

  Without this flag:

  [
    a,
    b,  #
    c
  ]
  

  With this flag:

  [
    a, b,  #
    c
  ]
  

  Before we had one flag that controlled two behaviors.

  • DISABLE_ENDING_COMMA_HEURISTIC=false (default):
    • Split a list that has a trailing comma.
    • Split a list that contains a comment.
  • DISABLE_ENDING_COMMA_HEURISTIC=true:
    • Don't split on trailing comma.
    • Don't split on comment.

  Now we have two flags.

... (truncated)

Commits

• 5cb90cb Bump version to v0.43.0
• 29db8b7 Actions(deps): Bump actions/checkout from 4.2.0 to 4.2.1 (#1249)
• 89d1cd0 Actions(deps): Bump peter-evans/create-pull-request from 6.0.2 to 7.0.5 (#1239)
• 7e21823 ylib2to3/pgen2/grammar: Fix Grammar.dump for parallel use with Grammar.load (...
• ed70540 pre-commit: Add hook for yapf-diff (#1246)
• 0de7684 Actions(deps): Bump actions/checkout from 4.1.2 to 4.2.0 (#1241)
• b951457 Fix CI (#1242)
• 1afbf71 Actions(deps): Bump actions/setup-python from 5.0.0 to 5.1.0 (#1218)
• fcd79bc Actions(deps): Bump peter-evans/create-pull-request from 6.0.0 to 6.0.2 (#1213)
• 507a900 Actions(deps): Bump actions/checkout from 4.1.1 to 4.1.2 (#1212)
• Additional commits viewable in compare view

You can trigger a rebase of this PR by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

[github-actions]

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

[claudevdm]

@damccorm do you know if this PR is still valid?

[damccorm]

@damccorm do you know if this PR is still valid?

It should be, updating this dep seems valid. Usually with dependabot I:

1. Check if the change is reasonable
2. Make sure we get green checks - in this case we have red statuses, but I can't tell if its because of flakes or not so I'll get dependabot to rebase/rerun checks
3. Merge if green
4. Close if not - I don't worry too much about this since dependabot will open a new pr for newer versions and we can also manually bump versions which need special attention

[damccorm]

@dependabot rebase

[github-actions]

Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:

R: @tvalentyn for label python.

Available commands:

• stop reviewer notifications - opt out of the automated review tooling
• remind me after tests pass - tag the comment author after tests pass
• waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

[jrmccluskey]

This PR actually creates a mismatch in yapf versions. Updating yapf in pyproject.toml changes the version users would require to build the project, but our linting/formatting workflows still specify 0.29.0 (which is why this PR didn't break those precommits.) We can either revert this PR for now, or we can fix the workflows + re-yapf the code-base. Thoughts @damccorm @tvalentyn ?

[damccorm]

Good call - thanks for catching this. Lets just revert for now and then we can bump yapf everywhere later if we want to.

[damccorm]

Actually, I see you already have a pr to do the upgrade. We can take that if it doesn't have issues

[jrmccluskey]

Threw together #33548 to see if that just works, will link workflow runs on that branch directly on the PR