Skip to content

python: Apply Envoy python-yapf formatting#2648

Merged
jpsim merged 1 commit intoenvoyproxy:mainfrom
phlax:python-yapf
Nov 1, 2022
Merged

python: Apply Envoy python-yapf formatting#2648
jpsim merged 1 commit intoenvoyproxy:mainfrom
phlax:python-yapf

Conversation

@phlax
Copy link
Member

@phlax phlax commented Nov 1, 2022

This applies the python-yapf formatting that Envoy uses

Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax
Copy link
Member Author

phlax commented Nov 1, 2022

(full disclaimer: im not a big fan of auto-formatters but this is the diff created by applying Envoy's formatter)

@jpsim
Copy link
Contributor

jpsim commented Nov 1, 2022

/retest

Copy link
Contributor

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

Fine to keep the formatter happy. Thanks!

@jpsim jpsim enabled auto-merge (squash) November 1, 2022 15:40
@jpsim jpsim merged commit 6537496 into envoyproxy:main Nov 1, 2022
jpsim added a commit that referenced this pull request Nov 4, 2022
…builder-function

* origin/main:
  remove the use of deprecated flag (#2658)
  Cronvoy: Map EM Errors to Cronet API Errors II (#1594) (#2633)
  build: revert boring patch (#2651)
  Bump Lyft Support Rotation (#2654)
  fix one issue blocking bumping Envoy (#2649)
  ci: remove Snow from Lyft EM rotation (#2650)
  ci: increasing timeouts (#2653)
  python: Apply Envoy python-yapf formatting (#2648)
  repo: Shellcheck cleanups (#2646)
  repo: Switch `pip_install` -> `pip_parse` (#2647)
  Use safe_malloc instead of new when creating new_envoy_map_entry (#2632)
  python: Pin requirement hashes (#2643)
  Disable flaky TestConfig.StringAccessors (#2642)
  ci: migrate from set-output to GITHUB_OUTPUT (#2625)
  Add a comment to addPlatformFilter (#2634)
  Allow Cronvoy to build with proguard. (#2635)
  Update Envoy (#2630)
  Add support for Platform and Native filters to C++ EngineBuilder (#2626)
  Register getaddrinfo in extension_registry (#2627)
  dns: stop using cares DNS resolver (#2618)

Signed-off-by: JP Simard <jp@jpsim.com>
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