Skip to content

Add Linter #8

@lee2sman

Description

@lee2sman

Currently L5 is written with extensively documented code. But there are extra lines, different spacing at times. Probably it should use a linter, especially as additional people hack on the codebase. Other potential benefits: catch bugs like undefined globals, typos, unused vars. Automates some aspect of feedback for contributors and makes clear style rules.

Describe the solution you'd like
Consider luacheck (install with luarocks). It seems the most widely used and can work with GitHub CI/CD.

Describe alternatives you've considered
The more build tools get added, the more complexit for 'getting started' on the codebase, so how much this benefits the project (or not) should be considered.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions