Skip to content

Typescript initial support - #144

Closed
igbopie wants to merge 9 commits into
ember-cli-code-coverage:masterfrom
igbopie:master
Closed

Typescript initial support#144
igbopie wants to merge 9 commits into
ember-cli-code-coverage:masterfrom
igbopie:master

Conversation

@igbopie

@igbopie igbopie commented Nov 14, 2017

Copy link
Copy Markdown

Hi!

I would like to add support to TS. My initial idea was to transpile TS to JS and pass that JS to BABEL using Babel-instrumenter. Then I realize that ember already knows how to build the code, why do we need to specify to coverage how to build the code? So I changed the addon hook to postBuild, that way I get the compiled src and the srcMaps. I read the sourceMaps and I use that to display the code.

This is a first approach. What do you think?

Thanks!

@igbopie

igbopie commented Nov 15, 2017

Copy link
Copy Markdown
Author

Yes, not all test are passing. The reason is that inline source map is not generated on those cases. I have to figure out how, or may don't even read src map in that case.

@rwjblue

rwjblue commented Nov 15, 2017

Copy link
Copy Markdown
Collaborator

In general, we should avoid landing large changes to master until after #141 has landed...

@igbopie

igbopie commented Nov 15, 2017

Copy link
Copy Markdown
Author

Sounds good!

@RobbieTheWagner

Copy link
Copy Markdown
Collaborator

@igbopie there was a pretty significant refactor recently. Would you mind reimplementing with that as the base?

Ignacio Bona Piedrabuena added 2 commits April 3, 2018 14:50
@RobbieTheWagner

Copy link
Copy Markdown
Collaborator

@igbopie I'm not sure what is going on here, but I think TS should be supported from @dfreeman's PR we just merged #164

@igbopie igbopie closed this Apr 5, 2018
@igbopie

igbopie commented Apr 5, 2018

Copy link
Copy Markdown
Author

I will take a look! Thank you

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.

4 participants