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

add simple lua format base on js. #1569

Closed
wants to merge 2 commits into from
Closed

add simple lua format base on js. #1569

wants to merge 2 commits into from

Conversation

hxsf
Copy link
Contributor

@hxsf hxsf commented Mar 28, 2017

What does this implement/fix? Explain your changes.

  1. I want to format Lua, but need install perl frist? It's not cool.
    so, I use js, and translate it to coffee.

  2. the current perl version has some bug like following:

  • it will change whitespace in string.
    local a = "a..b...c" will be local a = "a.b.c" (use '.' to represent ' ')

Does this close any currently open issues?

Of course, many of issues about 'Lua' and 'format'.

Any other comments?

yes, I am not good at coffee, so if my code is not friendly to read. just edit it.
BTW, when I use Travis CI, Lua is OK, but other 9 language fail, I do not think I'm causing it.

Checklist

Check all those that are applicable and complete.

  • Merged with latest master branch
  • Added examples for testing to examples/ directory
  • Travis CI passes (Mac support)
  • AppVeyor passes (Windows support)

@hxsf
Copy link
Contributor Author

hxsf commented Mar 29, 2017

wait, I found some thing wrong.

local a = -1 will be local a = - 1

I will fix it.

@hxsf hxsf closed this Mar 29, 2017
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.

1 participant