Skip to content

feat: Integrate LuaLS check in CI#7112

Open
mbergen wants to merge 54 commits intomainfrom
ci-luals
Open

feat: Integrate LuaLS check in CI#7112
mbergen wants to merge 54 commits intomainfrom
ci-luals

Conversation

@mbergen
Copy link
Copy Markdown
Collaborator

@mbergen mbergen commented Feb 18, 2026

Summary

Uses LuaLS' --check to check for type errors wrt to annotations in CI.

Only report errors in lua files changed in a PR.

How did you test this change?

Self-tested in actions

@mbergen mbergen added the CI/CD label Feb 18, 2026
@mbergen
Copy link
Copy Markdown
Collaborator Author

mbergen commented Feb 18, 2026

Debugging Github Workflows is so much fun 🙃

@mbergen mbergen marked this pull request as ready for review February 18, 2026 15:00
@mbergen mbergen requested review from a team as code owners February 18, 2026 15:00
@mbergen mbergen marked this pull request as ready for review March 11, 2026 17:59
@mbergen mbergen requested a review from ElectricalBoy March 11, 2026 18:07
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This change is mostly here for demonstrating the check on only changed files

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.

2 participants