Skip to content

Add and use method to check true/false values#274

Merged
sandain merged 1 commit intomainfrom
true_value
Mar 16, 2021
Merged

Add and use method to check true/false values#274
sandain merged 1 commit intomainfrom
true_value

Conversation

@sandain
Copy link
Member

@sandain sandain commented Jan 2, 2021

Add and use method to check true/false values. Inspired by PR #271 from @Roflicide

@zanix
Copy link
Member

zanix commented Jan 2, 2021

Nice, now hopefully I can find time to test this

@michael2847
Copy link
Member

yes, i will test this as well, thanks for the PR @sandain !

@sandain
Copy link
Member Author

sandain commented Jan 24, 2021

Has anyone had the chance to test this? I'm not having any issues with it, but I don't have time to play this game any more, so I haven't fully tested this change. All I have time to check for is whether the server starts and whether I get a valid query back from the server, and both work just fine.

@zanix
Copy link
Member

zanix commented Jan 24, 2021

Not yet, haven't had time

@michael2847
Copy link
Member

just tested this and it works nicely!

Copy link
Member

@zanix zanix left a comment

Choose a reason for hiding this comment

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

Seems to work, ship it!

@sandain
Copy link
Member Author

sandain commented Mar 16, 2021

Thanks for testing @Roflicide and @zanix

@sandain sandain merged commit 29d2107 into main Mar 16, 2021
@sandain sandain deleted the true_value branch March 31, 2021 01:28
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.

3 participants