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 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: pygments/pygments.rb#10

@travisbot
Copy link

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

@ajacksified
Copy link

Moonscript support would be super cool. 👍

@losinggeneration
Copy link

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

+1

@8bitmcu
Copy link

8bitmcu commented Nov 5, 2012

This would be great.

@egonSchiele
Copy link

+1

@chase
Copy link

chase commented Nov 5, 2012

I would really like to see this merged!

@GriffinSchneider
Copy link

👍

@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

another +1

@ajacksified
Copy link

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
@tnm tnm merged commit 1abcb2e into github-linguist:master Jan 8, 2013
@etandel
Copy link

etandel commented Jan 8, 2013

Great! 🤘

@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.