1
7
Fork
You've already forked debugger.lua
0

Fix: pass luacheck, make global functions local #74

Merged
lua-rocks merged 1 commit from patch-1 into master 2021年08月10日 21:36:49 +02:00
lua-rocks commented 2021年08月05日 13:59:11 +02:00 (Migrated from github.com)
Copy link

Most of these changes are not important and was made just to get rid of annoying luacheck warnings, but some functions in your code was declared globally, so this fix is not decorative :)

Most of these changes are not important and was made just to get rid of annoying luacheck warnings, but some functions in your code was declared globally, so this fix is not decorative :)
slembcke commented 2021年08月10日 21:39:19 +02:00 (Migrated from github.com)
Copy link

Oh gosh. I used to have a script in one of my projects that would throw errors whenever globals where defined. I guess I got complacent. Thanks!

I do get why people have different thoughts about whitespace, but I didn't merge those changes. Sorry if that's an inconvenience.

Oh gosh. I used to have a script in one of my projects that would throw errors whenever globals where defined. I guess I got complacent. Thanks! I do get why people have different thoughts about whitespace, but I didn't merge those changes. Sorry if that's an inconvenience.
lua-rocks commented 2021年08月11日 10:52:52 +02:00 (Migrated from github.com)
Copy link

Thank you!

Thank you!
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
slembcke/debugger.lua!74
Reference in a new issue
slembcke/debugger.lua
No description provided.
Delete branch "patch-1"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?