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.

Sign up for GitHub

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

Jump to bottom

Issue #417: support nil params on os.date() #420

Open

cezarguimaraes wants to merge 1 commit into yuin:master from cezarguimaraes:fix-os-date-nil

cezarguimaraes commented Dec 13, 2022

Fixes #417 .

Changes proposed in this pull request:

nil params passed to os.date are interpreted as the default values

couldn't use L.OptString because unlike L.CheckString, it does not convert numbers to strings and lua allows os.date(2) = "2"


          Issue yuin#417: support nil params on os.date()

5cb9dbd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment