search

luttje / glua-api-snippets

Scrapes the Garry's Mod Wiki in order to build Lua Language Server comments that will provide IDE suggestions and autocompletion.
MIT License
18 stars 6 forks source link

`include` and `IncludeCS` should not be treated as `require` #32

Closed b0mbie closed 8 months ago

b0mbie commented 8 months ago

This is how the page for include describes its first argument, fileName:

The name of the script to be executed. The path must be either relative to the current file, or be an absolute path (relative to and excluding the lua/ folder).

This argument is a path, which does not follow the require path format, where dots are separators. If include is treated like Lua 5.1's dofile, it would be correct in this case.

luttje commented 8 months ago

Edit: awesome! I just now noticed you made a PR as well, much appreciated 👍

luttje commented 8 months ago

@all-contributors add @b0mbie for bug

allcontributors[bot] commented 8 months ago

@luttje

I've put up a pull request to add @b0mbie! :tada: