Skip to content

Fix implicit string -> bool conversion#1705

Merged
RomanDzhabarov merged 1 commit intoenvoyproxy:masterfrom
tschroed:quote-true
Sep 20, 2017
Merged

Fix implicit string -> bool conversion#1705
RomanDzhabarov merged 1 commit intoenvoyproxy:masterfrom
tschroed:quote-true

Conversation

@tschroed
Copy link
Copy Markdown
Contributor

true == true
"true" == true

false == false
"false" == true

:)

string.

Signed-off-by: Trevor Schroeder <trevors@google.com>
@RomanDzhabarov RomanDzhabarov merged commit e9a5865 into envoyproxy:master Sep 20, 2017
jpsim pushed a commit that referenced this pull request Nov 28, 2022
…1705)

Description: Move the Java/Kotlin MacOS part of the android_test workflow to Engflow's remote execution and improve build times by ~10x

The remaining android_test Linux workflow requires some Java setup so it'll be done in a subsequent PR

Risk Level: Low
Testing: See android_build and android_test checks
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
…1705)

Description: Move the Java/Kotlin MacOS part of the android_test workflow to Engflow's remote execution and improve build times by ~10x

The remaining android_test Linux workflow requires some Java setup so it'll be done in a subsequent PR

Risk Level: Low
Testing: See android_build and android_test checks
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
)

**Description**

This is a draft introduced in #1699

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.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.

4 participants