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.

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

Fix issue #63 #64

Merged
merged 2 commits into from Sep 5, 2017
Merged

Fix issue #63 #64

merged 2 commits into from Sep 5, 2017

Conversation

fperrad
Copy link
Contributor

@fperrad fperrad commented Aug 28, 2017

coxpcall is only required with PUC Lua 5.1, but not with LuaJIT

see issue #63

@fperrad
coxpcall is only required with PUC Lua 5.1, but not with LuaJIT
37d2916 
use the same logic as in copas.lua.
this fixes issue #63.
@coveralls
Copy link

coveralls commented Aug 28, 2017 •
edited

Coverage Status

Coverage remained the same at 84.178% when pulling 37d2916 on fperrad:fix63 into 394c359 on keplerproject:master.

@Tieske
Copy link
Member

Tieske commented Aug 30, 2017

@fperrad would you mind adding a new "unreleased" block to the changelog here: https://github.com/keplerproject/copas/blob/master/doc/us/index.html#L117 ?

@coveralls
Copy link

coveralls commented Aug 30, 2017 •
edited

Coverage Status

Coverage remained the same at 84.178% when pulling 4fc65b6 on fperrad:fix63 into 394c359 on keplerproject:master.

@Tieske
Copy link
Member

Tieske commented Sep 5, 2017

thank you!

@Tieske Tieske merged commit 8d1b5a9 into lunarmodules:master Sep 5, 2017
@Tieske Tieske mentioned this pull request Sep 5, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@fperrad @coveralls @Tieske