Skip to content

Fix lua@latest resolving to all instead of the newest Lua release#10

Merged
yeshan333 merged 2 commits into
mainfrom
copilot/fix-lua-latest-installation
May 1, 2026
Merged

Fix lua@latest resolving to all instead of the newest Lua release#10
yeshan333 merged 2 commits into
mainfrom
copilot/fix-lua-latest-installation

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

lua@latest could resolve to the manifest’s all entry, which led the plugin to download lua-all.tar.gz and fail during install. This change makes latest resolve from real Lua release versions only.

  • Version resolution

    • Filter pseudo-versions such as all out of the available-version list exposed by the plugin.
    • Keep explicit manifest lookups unchanged so direct version installs continue to use the existing manifest data.
  • Regression coverage

    • Add a focused Linux E2E case for vfox install lua@latest so the latest path is exercised directly.
for line in string.gmatch(resp.body, '([^\n]+)') do
    local version, checksum = parse_version_line(line)
    if version and checksum and string.match(version, "^%d") then
        table.insert(result, {
            version = version,
            checksum = checksum
        })
    end
end

Copilot AI linked an issue May 1, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix installation issue with lua@latest version Fix lua@latest resolving to all instead of the newest Lua release May 1, 2026
Copilot AI requested a review from yeshan333 May 1, 2026 04:17
@yeshan333 yeshan333 marked this pull request as ready for review May 1, 2026 04:24
@gemini-code-assist
Copy link
Copy Markdown

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

@yeshan333 yeshan333 merged commit 6653a63 into main May 1, 2026
14 checks passed
@yeshan333 yeshan333 deleted the copilot/fix-lua-latest-installation branch May 1, 2026 05:21
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.

lua@all listed as latest and error on install

2 participants