Skip to content

chore: bump CI LuaRocks pin to 3.13.0 - #127

Draft
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1786957518-luarocks-3-13
Draft

chore: bump CI LuaRocks pin to 3.13.0#127
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1786957518-luarocks-3-13

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Requirements

  • I have added test coverage for new or changed functionality (CI-only change; existing test suite covers it)
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions (validated locally on Lua 5.3; CI runs 5.1/5.2/5.3/luajit-2.1)

Related issues

None — routine dependency audit of this repo and its examples.

Describe the solution you've provided

Bumps the exact LuaRocks pin used by CI and the install-lua-sdk workflow from 3.12.0 to 3.13.0 (latest LuaRocks release, published 2026-01-28). No shipped code or documented minimums change.

Audit result for the LaunchDarkly dependencies — no change needed:

Dependency Configured Latest
C++ Server-side SDK (.github/variables/cpp-sdk-versions.env, example Dockerfiles) 3.13.1 3.13.1
C++ Server-side SDK Redis Source 2.4.1 2.4.1

README "Minimum Version" entries (3.9.0 / 2.1.19) are deliberately left alone — the newer SDK works but is not required.

No deprecated LaunchDarkly C-binding APIs are used by launchdarkly-server-sdk.c / launchdarkly-server-sdk-redis.c; both compile warning-free against C++ server SDK 3.13.1.

Describe alternatives you've considered

Leaving the pin at 3.12.0 (it works); bumping the hello-nginx OpenResty base image (1.21.4.1-0-jammy, quite old) — left out as it is unrelated to this audit and higher risk.

Additional context

Validated locally on Ubuntu 22.04 / Lua 5.3, prebuilt C++ server SDK + Redis Source 2.4.1 dynamic Linux artifacts, Boost 1.81.0:

  • luarocks 3.13.0 built both rockspecs (launchdarkly-server-sdk, launchdarkly-server-sdk-redis) successfully.
  • luarocks test launchdarkly-server-sdk-2.2.0-0.rockspecRan 20 tests ... 20 successes, 0 failures (same result with 3.12-era luarocks and with 3.13.0).
  • Ran the hello-lua-server example against a real environment SDK key: it initialized, connected, and printed The sample-feature feature flag evaluates to false. (flag absent in that project, so the default was returned).

Link to Devin session: https://app.devin.ai/sessions/f15a4832f3d24aefaadc15789ad9d241

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

@cursor review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants