Skip to content

Add MoonScript#246

Merged
tnm merged 3 commits into
github-linguist:masterfrom
leafo:master
Jan 8, 2013
Merged

Add MoonScript#246
tnm merged 3 commits into
github-linguist:masterfrom
leafo:master

Conversation

@leafo

@leafo leafo commented Aug 30, 2012

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

@ajacksified

Copy link
Copy Markdown

Moonscript support would be super cool. 👍

@losinggeneration

Copy link
Copy Markdown

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

@etandel

etandel commented Oct 25, 2012

Copy link
Copy Markdown

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

@nilnor

nilnor commented Oct 29, 2012

Copy link
Copy Markdown

This would be very nice, please consider merging this!

@gotosleep

Copy link
Copy Markdown

+1

@8bitmcu

8bitmcu commented Nov 5, 2012

Copy link
Copy Markdown

This would be great.

@egonSchiele

Copy link
Copy Markdown

+1

@chase

chase commented Nov 5, 2012

Copy link
Copy Markdown

I would really like to see this merged!

@GriffinSchneider

Copy link
Copy Markdown

👍

@fillest

fillest commented Dec 8, 2012

Copy link
Copy Markdown

Yes please

@leafo

leafo commented Jan 8, 2013

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown

another +1

@ajacksified

Copy link
Copy Markdown

All the +1s

@tnm

tnm commented Jan 8, 2013

Copy link
Copy Markdown
Contributor

Can we get some sample files?

@leafo

leafo commented Jan 8, 2013

Copy link
Copy Markdown
Contributor Author

Sample added, anything else?

@tnm

tnm commented Jan 8, 2013

Copy link
Copy Markdown
Contributor

@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

leafo commented Jan 8, 2013

Copy link
Copy Markdown
Contributor Author

Okay, should be set. Thanks!

@tnm

tnm commented Jan 8, 2013

Copy link
Copy Markdown
Contributor

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

etandel commented Jan 8, 2013

Copy link
Copy Markdown

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.