-
Notifications
You must be signed in to change notification settings - Fork 104
feat: Integrate LuaLS check in CI #7112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
mbergen
wants to merge
54
commits into
main
Choose a base branch
from
ci-luals
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
fb63853
Integrate LuaLS setup and check in workflow
mbergen f39cd62
Update LuaLS logging in luacheck workflow
mbergen 684a38c
Update LuaLS check to specify target directory
mbergen b7f6a37
Use config?
mbergen 68b7acc
Update library paths in .luarc.json
mbergen 61e5edf
Install 'busted' alongside 'luacheck' in workflow
mbergen 4d88e4b
install command
mbergen 9ef8054
Revert: Update library paths in .luarc.json
mbergen bbb9090
Adjust
mbergen 07c6b0e
Set env 3rd?
mbergen 3e5b391
Fix output verbosity in LuaLS setup and update path
mbergen 8d18f17
Add CI library paths to .luarc.json
mbergen 7b12374
Remove CI comment from .luarc.json
mbergen 9cbca45
Update library paths in .luarc.json
mbergen 8321402
Simplify LuaLS check command
mbergen 0e5eac4
Remove busted installation from luacheck workflow
mbergen 781fdcf
Add 'lua/spec' to workspace.ignoreDir
mbergen 6d6e06f
Update LuaLS command to use config path
mbergen 3733e9f
Revert changes to .luarc.json
mbergen 3f34d22
Update .luarc.json
mbergen 6f2b5a0
Add LuaLS report upload step in workflow
mbergen 95dbfe2
Refactor LuaLS setup and checks in workflow
mbergen ff076d6
Update luacheck.yml
mbergen e46f2a0
Checkout
mbergen 5f0ab82
check parameter should be workspace root
mbergen 582edba
Install LuaLS into /usr/bin
mbergen 2aef059
~/.local instead
mbergen 3cb84ea
Issue warnings
mbergen a735967
Set pattern, use tee, readability
mbergen e704303
Update luacheck.yml
mbergen e61ae69
Pattern as env
mbergen 0110a9f
echo pattern for test
mbergen 38a2719
Specify shell
mbergen d0f8e03
debug output
mbergen 236fd58
Adjust pattern for injected colors
mbergen 41b36bf
Update luacheck.yml
mbergen b6bf340
Use sed
mbergen a5c0e25
Use temp file
mbergen 2e43d31
Continue on fail
mbergen bf3b046
upload result
mbergen 2fd9ba3
Produce summary
mbergen 130a22e
Merge branch 'main' into ci-luals
mbergen c064bc2
Use changed-files
mbergen 367dd47
Change lua file with warnings
mbergen c8541ce
Check only changed
mbergen 89fc417
Change lua file again
mbergen 01fd343
Update luacheck.yml
mbergen 99dd6e4
Update luacheck.yml
mbergen 220eec3
change file
mbergen d534690
show grep output
mbergen dfbe4c5
Update luacheck.yml
mbergen 2d53d27
Update luacheck.yml
mbergen 625204c
Invert output
mbergen e5f8b92
Merge branch 'main' into ci-luals
mbergen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
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