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

Transition is_true to is_truthy to clarify that the test is not based on identity with or equality to the True singleton. #3657

Merged
merged 1 commit into from
Dec 16, 2023

Conversation

adamreichold
Copy link
Member

Based on the discussion in #3638 (comment) and following.

@adamreichold adamreichold force-pushed the is-truthy branch 2 times, most recently from a1aecd3 to 20a7fa7 Compare December 16, 2023 11:16
Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

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

Thanks! Just one tiny typo!

src/instance.rs Outdated Show resolved Hide resolved
… on identity with or equality to the True singleton.
@adamreichold adamreichold added this pull request to the merge queue Dec 16, 2023
Merged via the queue into main with commit ff50285 Dec 16, 2023
36 of 37 checks passed
@adamreichold adamreichold deleted the is-truthy branch December 16, 2023 13:53
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