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

java: run clang-format for Java files #55

Closed
rebello95 opened this issue Jun 7, 2019 · 2 comments
Closed

java: run clang-format for Java files #55

rebello95 opened this issue Jun 7, 2019 · 2 comments
Assignees
Labels
ci platform/android Issues related to Android

Comments

@rebello95
Copy link
Contributor

clang-format can be used for auto-formatting Java files the same way it is for Objective-C and C++.

@rebello95 rebello95 added platform/android Issues related to Android ci labels Jun 7, 2019
@rebello95 rebello95 added this to the v0.1 "Antipasto" milestone Jun 7, 2019
@rebello95 rebello95 self-assigned this Jun 7, 2019
@rebello95
Copy link
Contributor Author

Depends on:
envoyproxy/envoy#7210
envoyproxy/envoy#7211

@rebello95
Copy link
Contributor Author

#56

rebello95 added a commit that referenced this issue Jun 10, 2019
`clang-format` can be used for auto-formatting Java files the same way it is for Objective-C and C++.

2 spaces were selected for spacing to maintain parity with upstream.

Depends on:
envoyproxy/envoy#7210
envoyproxy/envoy#7211

#55

Signed-off-by: Michael Rebello <[email protected]>
jpsim pushed a commit to envoyproxy/envoy that referenced this issue Nov 28, 2022
`clang-format` can be used for auto-formatting Java files the same way it is for Objective-C and C++.

2 spaces were selected for spacing to maintain parity with upstream.

Depends on:
#7210
#7211

envoyproxy/envoy-mobile#55

Signed-off-by: Michael Rebello <[email protected]>
Signed-off-by: JP Simard <[email protected]>
jpsim pushed a commit to envoyproxy/envoy that referenced this issue Nov 29, 2022
`clang-format` can be used for auto-formatting Java files the same way it is for Objective-C and C++.

2 spaces were selected for spacing to maintain parity with upstream.

Depends on:
#7210
#7211

envoyproxy/envoy-mobile#55

Signed-off-by: Michael Rebello <[email protected]>
Signed-off-by: JP Simard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci platform/android Issues related to Android
Projects
None yet
Development

No branches or pull requests

1 participant