Skip to content

Add support for Lua code blocks#132

Merged
jacobdeichert merged 4 commits into
jacobdeichert:masterfrom
yilisharcs:lua-block
Jan 10, 2026
Merged

Add support for Lua code blocks#132
jacobdeichert merged 4 commits into
jacobdeichert:masterfrom
yilisharcs:lua-block

Conversation

@yilisharcs
Copy link
Copy Markdown
Contributor

Which issue does this fix?

Closes #131

Describe the solution

Added a new executor for lua, taking from the ruby executor.

@jacobdeichert
Copy link
Copy Markdown
Owner

Thanks for the PR, I plan on reviewing/testing this week.

@jacobdeichert
Copy link
Copy Markdown
Owner

jacobdeichert commented Jan 9, 2026

Hey there, I haven't forgotten this.. just been busy.

Before I approve and merge this, I need to fix CI and hope to do so this weekend.

In the meantime, can you add a Lua code block example to the README like I did in this PR ? If not, no worries.. I can do that post-merge.

.success();
}

#[test]
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Just 2 changes to get Lua running under GH Actions CI

  1. Above this line, add this: #[cfg(target_os = "linux")]
  2. Update ci.yml to match my changes here: https://github.com/jacobdeichert/mask/compare/test-lua-runtime

After that, I'll approve and merge!

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Also please pull in latest from master. I fixed CI so your PR should pass after all of these changes are in place.

Copy link
Copy Markdown
Owner

@jacobdeichert jacobdeichert left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the contribution!

@jacobdeichert jacobdeichert merged commit e4f2387 into jacobdeichert:master Jan 10, 2026
7 checks passed
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.

Add support for lua code blocks

2 participants