Skip to content

Review and edit Lua docstrings#2717

Open
oznogon wants to merge 10 commits intodaid:masterfrom
oznogon:fix-script-docs
Open

Review and edit Lua docstrings#2717
oznogon wants to merge 10 commits intodaid:masterfrom
oznogon:fix-script-docs

Conversation

@oznogon
Copy link
Contributor

@oznogon oznogon commented Feb 13, 2026

  • Fix several examples that use a different function name than the one being documented (setSystemCoolant using setSystemPowerFactor, setPlanetAtmosphereTexture using setPlanetSurfaceTexture, several getters and setters mixed up)
  • Align examples of function usage to the arguments they actually use
  • Fix some incorrect defaults
  • Fix incorrect ship system heat, power, and health ranges
  • Remove extraneous arguments from getters (getHeading(0))
  • Remove extraneous colons from example function calls
  • Add missing docstring for VisualAsteroid
  • Add missing example to ModelData:setNormalMap
  • Flag setBeamFireSound/setBeamFireSoundPower as not implemented
  • Fix chaining order in ScanProbe example
  • Fix incorrect return type on getWaypointID
  • Fix incorrect scan state values in examples (full not fullscan)
  • Fix claim of case-insensitivity in ScienceDatabase set/getKeyValue, removeKey
  • Use "entity" or "ship" in place of legacy class names like SpaceObject or SpaceShip
  • Remove legacy class names in descriptions where they no longer apply
  • Remove duplicated onPickUp description and neutralize references to Artifacts and SupplyDrops
  • Remove manual angle behavior described in commandFireTube (needs verification)
  • Replace references to "meters" with "game units"
  • Fix several typos ("netural", "electical", "consideres", etc.)
  • Fix several references to incorrect source implementations (point to components/systems instead of spaceshipParts)

@oznogon oznogon marked this pull request as ready for review February 15, 2026 01:49
pickup={}
}
for k, v in pairs(__model_data["ammo_box"]) do
if string.sub(1, 2) ~= "__" then
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least you copied my mistake :P

@oznogon oznogon requested a review from daid February 26, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants