Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

bit library: add Lua 5.4 #174

Closed
wants to merge 1 commit into from
Closed

bit library: add Lua 5.4 #174

wants to merge 1 commit into from

Conversation

jprjr
Copy link

@jprjr jprjr commented Nov 12, 2020

No description provided.

@coveralls
Copy link

coveralls commented Nov 12, 2020 •
edited
Loading

Coverage Status

Coverage remained the same at 87.781% when pulling d1e8848 on jprjr:master into 47225d0 on daurnimator:master.

@jprjr
Copy link
Author

jprjr commented Nov 23, 2020

You know what, I think I'm going to redo this.

Right now, anytime a new version of Lua comes out, you'd have to add another check.

I'll instead wrap doing a load + pcall, that way it should be future-proof

@jprjr jprjr mentioned this pull request Nov 23, 2020
Closed
@jprjr jprjr closed this Nov 23, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@daurnimator daurnimator daurnimator approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jprjr @coveralls @daurnimator