Skip to content

Support typescript-mode #274

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

Merged
merged 1 commit into from
May 3, 2018
Merged

Support typescript-mode #274

merged 1 commit into from
May 3, 2018

Conversation

ybiquitous
Copy link
Contributor

This fixes #139.

For more details, see https://github.com/ananthakumaran/typescript.el.

@AndreaCrotti
Copy link
Owner

Hi @ybiquitous thanks for your contribution, but just to clarify, doesn't it just mean that all the JS snippets will be available to typescript mode with this change?
It doesn't really add any specific typescript specific snippets right?

@AndreaCrotti AndreaCrotti added the seen PR was seen and initial feedback was given label May 3, 2018
@ybiquitous
Copy link
Contributor Author

Hi. @AndreaCrotti
As you pointed out, this PR does not add TypeScript specific snippets.

However, since TypeScript is a superset of JavaScript, all JavaScript snippets can also be used in TypeScript.

I wanted to use the TypeScript snippet with minimal effort first, so I just simply inherit JavaScript snippets in this PR.

@AndreaCrotti
Copy link
Owner

Yes sure fine, makes sense 👍

@AndreaCrotti AndreaCrotti merged commit 214ddf7 into AndreaCrotti:master May 3, 2018
@ybiquitous
Copy link
Contributor Author

Thank you for understanding! 😸

@ybiquitous ybiquitous deleted the support-typescript-mode branch May 3, 2018 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
seen PR was seen and initial feedback was given
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add typescript snipets
2 participants