Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Set oid.selfFreeing to true instead of "true"#1033

Merged
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
smith-kyle:string-to-booleansmith-kyle/nodegit:string-to-booleanCopy head branch name to clipboard
May 26, 2016
Merged

Set oid.selfFreeing to true instead of "true"#1033
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
smith-kyle:string-to-booleansmith-kyle/nodegit:string-to-booleanCopy head branch name to clipboard

Conversation

@smith-kyle
Copy link
Contributor

Does not modify code generation since previously the string "true" evaluated to truthy

Does not modify code generation since previously the string "true" evaluated to truthy
@tbranyen
Copy link
Member

Ah okay good catch. This could have caused problems if we were using ===

@johnhaley81
Copy link
Collaborator

Thanks @smith-kyle

@johnhaley81 johnhaley81 merged commit a86dc6f into nodegit:master May 26, 2016
@smith-kyle smith-kyle deleted the string-to-boolean branch October 14, 2016 14:44
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.