-
Notifications
You must be signed in to change notification settings - Fork 60
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
LuaLS Annotations + Documentation #443
base: master
Are you sure you want to change the base?
Conversation
Wire team will tell you is a waste of time. I had only one patch like this ever merged :D but good effort nonetheless! |
@@ -1272,7 +1291,7 @@ local function AdvDupe2_Spawn() | |||
end | |||
|
|||
AdvDupe2.SpawningEntity = true | |||
local Ent = CreateEntityFromTable(v, Queue.Player) |
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.
Why remove this local? Please make sure there's no code changes are in this PR if you're going to add a ton of function comments.
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.
I have no clue how this happened.
These are useful additions. Your PRs had issues because you always wanted to include a bunch of cleanup or style changes along with your bugfixes. |
@Denneisk you never marked this ready for review. Is it? |
I forgot about this lol |
Annotates everything with LuaLS annotations, providing critical documentation that would otherwise require carefully tracing the execution flow of AdvDupe2.
I hope that future contributors will be able to benefit from my progressive understanding of AdvDupe2.
Let me know if this is a waste of time before I keep going, and if any changes need to be made.