forked from mpeterv/cluacov
-
Notifications
You must be signed in to change notification settings - Fork 5
support Lua 5.5 #7
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
Merged
Merged
+1,244
−62
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* install the latest hererocks directly from its repository * also install luacov and busted directly from their repositories
* install the latest hererocks directly from its repository * also install luacov and busted directly from their repositories
* install luacov and busted directly from their repositories
Tieske
reviewed
Jan 9, 2026
Tieske
reviewed
Jan 9, 2026
Tieske
reviewed
Jan 9, 2026
* also replace keplerproject with luarocks account
* also removed AppVeyor and Travis CI providers
Author
|
following your advices, the recent changes were:
|
Tieske
reviewed
Jan 12, 2026
Co-authored-by: Thijs Schreijer <[email protected]>
Author
|
@Tieske applied your suggestion |
This was referenced Jan 15, 2026
daurnimator
approved these changes
Jan 21, 2026
Member
|
Anyone know what the process is to do a release for this repo? |
Author
|
I guess Tieske is waiting for Hisham to move this repo to LunarModules account (see #8), together with the registry on LuaRocks org website |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Support Lua 5.5
Important
I do not have any experience with this module (
cluacov), I just saw @Tieske CI failing onluacovdue this module and tried to update it to give a help. Since the tests on my branch were all passing, I decided to open this PR.Changes
Added Lua 5.5.0
llimits.handlobject.hheadersMinimal additions and edits on
deepactivelines.cto account for Lua 5.5Yet on
deepactivelines.cfile, the code for Lua 5.5 required a minimal change regarding the fieldis_vararg. Such field cannot be found on Proto struct anymore. Instead, Roberto provided a macro calledisvararg, which serves the same purposeFor AppVeyor and Travis CI providers, the latest hererocks is installed directly from its repository. This change was done to be able to install Lua 5.5 and LuaRocks 3.13.0;
Following the idea of AppVeyor that installs
luacovdirectly from its repositorycluacov/appveyor.yml
Lines 21 to 22 in e81e0f9
now each CI provider is also installing
luacovfrom its repo.bustedfrom its repo, because the version hosted on LuaRocks had a change in the control variable that was only addressed at fix(lua5.5): add local variable lunarmodules/busted#760.Notes