Skip to content
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

Annotate engine/Sim batch 4 #4056

Merged

Conversation

Hdt80bro
Copy link
Contributor

@Hdt80bro Hdt80bro commented Jul 16, 2022

IEffect and Unit
Also includes some changes in #4052 to comprehensively annotate Unit.lua (merge that one first)

@Hdt80bro Hdt80bro marked this pull request as draft July 16, 2022 21:17
@Hdt80bro
Copy link
Contributor Author

Nice work @Garanas !

@Garanas
Copy link
Member

Garanas commented Jul 23, 2022

It appears some of them are invalid - I've asked the engine devs 😄

All the current entries are directly from the engine and appear
to work as expected
@Garanas
Copy link
Member

Garanas commented Jul 23, 2022

All of these are valid 👍

@Hdt80bro Hdt80bro marked this pull request as ready for review July 24, 2022 05:03
@Hdt80bro Hdt80bro added this to In progress in Annotation Jul 25, 2022
@Garanas Garanas merged commit 6487aa5 into FAForever:deploy/fafdevelop Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Annotation
In progress
Development

Successfully merging this pull request may close these issues.

None yet

2 participants