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

feat(config): add support for TypeScript #2224

Merged
merged 1 commit into from
Jul 3, 2016

Conversation

wesleycho
Copy link
Member

  • Add TypeScript support for karma.conf.ts

This should address #1727.

} else if (fs.existsSync('./.config/karma.conf.js')) {
configFile = './.config/karma.conf.js'
} else if (fs.existsSync('./.config/karma.conf.coffee')) {
configFile = './.config/karma.conf.coffee'
} else if (fs.existsSync('./config/karma.conf.ts')) {
Copy link
Contributor

@maksimr maksimr Jul 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here should be .config not config

@dignifiedquire
Copy link
Member

Would be good to have some test for this.

@wesleycho
Copy link
Member Author

Any tip as to where to look for putting a test? I didn't see anything for coffeescript or livescript, and the way they are set up, it may not be so simple to set up since support is checked globally.

@wesleycho wesleycho force-pushed the feat/typescript branch 3 times, most recently from f5a8841 to ece7d03 Compare July 2, 2016 05:43
- Add TypeScript support for `karma.conf.ts`
@wesleycho
Copy link
Member Author

Found it, added a test. I also added a config template and code to the formatters to mirror the coffeescript and livescript code, although it seems they are not used.

@dignifiedquire
Copy link
Member

Thanks :octocat:

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.

3 participants