Add TypeScript recipe

[SamVerschueren]

I kept it simple. Feel free to provide feedback if I missed something or if I have to add some extra things.

@ivogabe Something you would like to add or improve?

[sindresorhus]

test(async t => {

[SamVerschueren]

Doesn't work. See microsoft/TypeScript#4885 (comment)

[sindresorhus]

Can you also add a link to the readme?

[ivogabe]

Can you also add outDir? That will compile the sources to a different directory. If you place all files in the same directory, you can easily confuse JS fileswith TS files.

[SamVerschueren]

Not sure about the outDir. AVA looks for test.js or test/*.js if I'm not mistaken. So what should I use as value for outDir then?

[ivogabe]

I have used lib/test for the TS tests and dist/test for the compiled tests. The options should be "outDir": "dist", "rootDir": "lib".

AVA needs some configuration for that, I started AVA using ava dist/test. It can also be configured in the readme. Though maybe dist/test/*.js could be added as a new test location?

[SamVerschueren]

Isn't it quite specific per project? I mean, some people will use TypeScript for the entire project, some might only use it for the tests. I don't want to force them in using a specific approach or directory structure. This recipe is just for the basics, they can always go from here and do some more advanced stuff with it, no?

[ivogabe]

👍 after you've addressed my two minor comments

[SamVerschueren]

@sindresorhus What link? Link to the AVA readme? Can't find something similar in the other recipes so guess I'm a little confused :p.

[sindresorhus]

@SamVerschueren Add it here: https://github.com/sindresorhus/ava#recipes

[SamVerschueren]

Damnit, just screwed up squasing my commits. Let me try to fix it.

[SamVerschueren]

Alright, don't gonna touch the commits anymore :). Not sure if I should add the outDir property to tsconfig.json?

@sindresorhus tips for squashing when you had to sync with the remote in between? It totally got screwed up the previous time :).

[ivogabe]

TypeScript users should already know how to use the outDir option, so it should be fine without it too.

[SamVerschueren]

We can always improve depending on the feedback via twitter/gitter/issues.

[sindresorhus]

tips for squashing when you had to sync with the remote in between?

Don't sync with git pull, but with git pull --rebase.

[sindresorhus]

But don't worry. I'll squash when I merge.

[SamVerschueren]

@sindresorhus Alright, thanks for the tip :)!

[sindresorhus]

Landed. I'd like to keep it simple as is. TypeScript will (should) know how to use the TSC options. Not the correct place to teach it here.

Thanks Sam!

[SamVerschueren]

Glad I could help :)

[ivogabe]

👍

[kasperlewau]

Don't sync with git pull, but with git pull --rebase.

off topic

I'd like to chime in on this statement and say; put it in your global git config. Fire and forget. Be rid of any nonsense merge [remote]master into [local]X commits.

# git >= 1.7.9
git config --global pull.rebase true

# git < 1.7.9 
git config --global branch.autosetuprebase always

I for one can't think of a time and place where a straight up git pull would be preferential. Sorry for the off topic comment!

[SamVerschueren]

Thanks @kasperlewau . Will save me some headaches in the future :).