JohnnyMorganz/StyLua

Support FiveM Lua syntax

Gellipapa opened this issue · 13 comments

I've recreated the issue because now the full-moon has been rewritten.

@JohnnyMorganz Hi!
I recently asked the creator of full-moon about when the parse rewrite will be done and he said that it will be done before the end of the day. I checked the next day and it was already merged.

I would like some help on how I can ?. and ` manage it using stylua and full-moon.

Thank you very much for your reply in advance.

Hey, sure. I will look into adding fivem Lua support in full-moon soon

Hey, sure. I will look into adding fivem Lua support in full-moon soon

Hi! Thanks for the quick reply.

If you could send me some description how to include such a charachter treatment I would be glad to have a look.

I would also be interested in implementation but thanks a lot if you do it.

@JohnnyMorganz Hi! Can you give me some status? Have you managed to implement the logic in full-moon? Thanks for the answer in advance.

Hey, no progress unfortunately. I haven't had the chance to work on full-moon yet. We still need to also update full-moon in stylua too, but it currently has some stack overflow issues that require fixing.

Do you have a list of all the Lua differences introduced in FiveM lua?

Hey, no progress unfortunately. I haven't had the chance to work on full-moon yet. We still need to also update full-moon in stylua too, but it currently has some stack overflow issues that require fixing.

Do you have a list of all the Lua differences introduced in FiveM lua?

It's okey. I don't have a list, unfortunately, but I know about the ones that work in the FiveM lua part. ? ` += -=
Thanks.

I think most of these should work already in stylua due to overlap with the Luau syntax.

What code is currently failing to format with stylua?

I think most of these should work already in stylua due to overlap with the Luau syntax.

What code is currently failing to format with stylua?

` and ? these characters.

image

https://github.com/esx-framework/esx_core/blob/main/%5Bcore%5D/esx_multicharacter/server/main.lua#L173

This was one of the errors that caused the stylua not to run.

@JohnnyMorganz Hi! Sorry for ping. Can you give me some status on this? Thanks.

@JohnnyMorganz Hi! What's up? Is there anything we can do to manage these characters?

Very sorry, I just have not had a chance to work on this project recently. It is still on my (somewhat large) TODO list

@JohnnyMorganz Hi! I saw the 2.0 update that you see in the near future this option to ignore these character cases?

Thanks for your reply.

Hey, sorry once again for the delay. Yes I think it shouldn't be too hard to add this now that the new syntax support was added. I am going to aim to solve this for the next release

For my own reference, the extended syntax is documented at https://github.com/citizenfx/lua/blob/luaglm-dev/cfx/README.md#power-patches