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

Make tonumber work on true and false #3112 #3113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davidfetter
Copy link
Contributor

As title and bug report.

["1", "invalid", "3", 4]
[1, 3, 4]
["1", "invalid", "3", 4, true, false]
[1, 3, 4, 1, 0]
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm just a random passerby in this repo, but having spent an obscene amount of time trying (failing) to get doc-building working yesterday (on Windows, long story), I think what you need to do here is make the change to docs/content/manual/manual.yml:2539 and then rebuild the docs per docs/README.md.

Maybe some words and another example in the tonumber section of the manpage might be nice, too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Getting that patch modified as you've suggested.

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