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

LSL support #1254

Merged
merged 1 commit into from
Feb 25, 2013
Merged

LSL support #1254

merged 1 commit into from
Feb 25, 2013

Conversation

nemurimasu
Copy link
Contributor

I've created a syntax mode for Linden Scripting Language based on C/C++ mode and the LSL Wiki as of 2013-01-27.

This is my first time making a syntax module for Ace. Let me know if I've done anything wrong.

based on C/C++ mode and the LSL Wiki as of 2013-01-27
@nightwing
Copy link
Member

Looks good. I'd be happy to merge but unfortunately we had to restore requirement for CLA,
so i have to ask, can you send one to [email protected] please. See https://github.com/ajaxorg/ace/blob/master/CONTRIBUTING.md for details.

@nemurimasu
Copy link
Contributor Author

I would like to remain fairly anonymous. I have a crazy plan to make something really cool (if I manage to work on it enough) but I'm afraid to link this identity to myself because I don't want to deal with people thinking I'm weird. I don't need to see this merged into the main Ace repository but thought it could be nice for somebody.

@nightwing
Copy link
Member

Ok, looks like in rare cases we can add modes without a cla since they are selfcontained, and do not affect the rest of the code (can be removed or replaced if needed). But majority of ace users don't care about cla so this is still useful.
Thanks for the pr, it's already something really cool!

nightwing added a commit that referenced this pull request Feb 25, 2013
@nightwing nightwing merged commit ad1e740 into ajaxorg:master Feb 25, 2013
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.

2 participants