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 MoonScript #246

Merged
merged 3 commits into from Jan 8, 2013
Merged

Add MoonScript #246

merged 3 commits into from Jan 8, 2013

Conversation

@leafo
Copy link
Contributor

leafo commented Aug 30, 2012

Adds MoonScript to the list of languages.

I didn't add a code sample because I can't run the script to regenerate the database because ruby pygments does not run on my system due to old version of rubypython: tmm1/pygments.rb#10

@travisbot
Copy link

travisbot commented Aug 30, 2012

This pull request passes (merged 0ff50a6 into 6ec907a).

@ajacksified
Copy link

ajacksified commented Oct 25, 2012

Moonscript support would be super cool. 👍

@losinggeneration
Copy link

losinggeneration commented Oct 25, 2012

Being such a simple addition, why hasn't this been merged?

@etandel
Copy link

etandel commented Oct 25, 2012

There aren't many of us, but we do exist! Support for MoonScript would be awesome!

@nilnor
Copy link

nilnor commented Oct 29, 2012

This would be very nice, please consider merging this!

@gotosleep
Copy link

gotosleep commented Nov 4, 2012

+1

@8bitmcu
Copy link

8bitmcu commented Nov 5, 2012

This would be great.

@egonSchiele
Copy link

egonSchiele commented Nov 5, 2012

+1

@chase
Copy link

chase commented Nov 5, 2012

I would really like to see this merged!

@GriffinSchneider
Copy link

GriffinSchneider commented Nov 5, 2012

👍

@fillest
Copy link

fillest commented Dec 8, 2012

Yes please

@leafo
Copy link
Contributor Author

leafo commented Jan 8, 2013

It's been 4 months since opening this. Is there anything I can do to move this along?

The pygments support is there and it already works fine in any markdown text:

class Thing
  name: "unknown"

class Person extends Thing
  say_name: => print "Hello, I am", @name

with Person!
  .name = "MoonScript"
  \say_name!

I'm not sure what is holding this back but it would be a great to have files highlighted as well.

Thanks

@egonSchiele
Copy link

egonSchiele commented Jan 8, 2013

another +1

@ajacksified
Copy link

ajacksified commented Jan 8, 2013

All the +1s

@tnm
Copy link
Contributor

tnm commented Jan 8, 2013

Can we get some sample files?

@leafo
Copy link
Contributor Author

leafo commented Jan 8, 2013

Sample added, anything else?

@tnm
Copy link
Contributor

tnm commented Jan 8, 2013

@leafo looks good, but you can actually drop the extra

extensions:
  - .moon

part. I'm in process of removing it from everywhere else. Then this should be good to go.

@leafo
Copy link
Contributor Author

leafo commented Jan 8, 2013

Okay, should be set. Thanks!

@tnm
Copy link
Contributor

tnm commented Jan 8, 2013

Seems fine.

tnm added a commit that referenced this pull request Jan 8, 2013
Add MoonScript
@tnm tnm merged commit 1abcb2e into github:master Jan 8, 2013
1 check passed
1 check passed
default The Travis build passed
Details
@etandel
Copy link

etandel commented Jan 8, 2013

Great! 🤘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

You can’t perform that action at this time.