Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update bazel readme for clang-format-8 on mac #6660

Merged
merged 2 commits into from
Apr 20, 2019

Conversation

dschaller
Copy link
Member

Signed-off-by: Derek Schaller dschaller@lyft.com

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md

Description: update bazel README to use clang-format from homebrew on mac
Risk Level: Low
Testing: Tested this configuration on my local machine
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Derek Schaller <dschaller@lyft.com>
mattklein123
mattklein123 previously approved these changes Apr 20, 2019
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123
Copy link
Member

@dschaller sorry needs master merge.

/wait

Signed-off-by: Derek Schaller <dschaller@lyft.com>
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit 2ae3322 into envoyproxy:master Apr 20, 2019
mpuncel added a commit to mpuncel/envoy that referenced this pull request Apr 22, 2019
* master:
  thread: remove ThreadFactorySingleton (envoyproxy#6658)
  router: support offseting downstream provided grpc timeout (envoyproxy#6628)
  router: defer per try timeout until downstream request is done (envoyproxy#6643)
  update bazel readme for clang-format-8 on mac (envoyproxy#6660)
  Implement some TODOs in quic_endian_impl.h (envoyproxy#6644)
  docs: add aspell to mac dependencies to fix check format script (envoyproxy#6661)
  config: fix delta xDS's use of (un)subscribe fields, more explicit protocol spec (envoyproxy#6545)

Signed-off-by: Michael Puncel <mpuncel@squareup.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