Skip to content

fix(lua): fix lua test and improve testing workflows#293

Merged
Peefy merged 1 commit into
kcl-lang:mainfrom
f4z3r:test/lua
May 3, 2026
Merged

fix(lua): fix lua test and improve testing workflows#293
Peefy merged 1 commit into
kcl-lang:mainfrom
f4z3r:test/lua

Conversation

@f4z3r
Copy link
Copy Markdown
Contributor

@f4z3r f4z3r commented May 3, 2026

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

  • N
  • Y

See: kcl-lang/kcl#388

2. What is the scope of this PR (e.g. component or file name):

Fix the Lua test and add workflows to ensure correct testing in the pipeline.

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features
  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Other

The current test did not contain valid Lua code. Moreover, it does not have the correct structure to be picked up by busted. I also updated the rockspec to ensure the package is properly packaged when it will be released.

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

  • N
  • Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

  • Unit test
  • Integration test
  • Benchmark (add benchmark stats below)
  • Manual test (add detailed scripts or steps below)
  • Other

Signed-off-by: Jakob Beckmann <f4z3r-github@pm.me>
@Peefy Peefy merged commit 53a69b2 into kcl-lang:main May 3, 2026
8 of 9 checks passed
@f4z3r f4z3r deleted the test/lua branch May 3, 2026 15:30
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.

2 participants