Skip to content

chore(ci): fix ci bump outdated dependencies#155

Merged
EvandroLG merged 2 commits intoEvandroLG:masterfrom
Tieske:fix/ci
Mar 7, 2026
Merged

chore(ci): fix ci bump outdated dependencies#155
EvandroLG merged 2 commits intoEvandroLG:masterfrom
Tieske:fix/ci

Conversation

@Tieske
Copy link
Contributor

@Tieske Tieske commented Mar 4, 2026

The tests pass on Lua 5.5. The only problem is that luacheck gets installed, which depends on argparse, which is not Lua 5.5 ready (see luarocks/argparse#35)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates GitHub Actions CI to support Lua 5.5 and modernize the workflow dependencies to address compatibility issues with older tooling.

Changes:

  • Bump runners from ubuntu-20.04 to ubuntu-22.04.
  • Expand test matrix to include Lua 5.5.
  • Switch Lua/LuaRocks setup actions and pin LuaRocks version.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/unix_build.yml Updates runner, adds Lua 5.5 to the matrix, and changes Lua/LuaRocks setup actions + LuaRocks version pinning.
.github/workflows/lint.yml Updates runner and changes Lua/LuaRocks setup actions + LuaRocks version pinning.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Tieske
Copy link
Contributor Author

Tieske commented Mar 7, 2026

@EvandroLG updated, all versions pinned now

@EvandroLG EvandroLG merged commit b9df51e into EvandroLG:master Mar 7, 2026
6 of 7 checks passed
@Tieske Tieske deleted the fix/ci branch March 7, 2026 16:14
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.

3 participants