Closed divido closed 8 years ago
Looks fine to me. I'll just let Jimi check it too - he's done most of the work on the Karma part of the project.
Hi @divido,
Sorry for not coming back to you earlier. Your pull request regarding the karma.included option seems good. The esprima upgrade, however, I would like to see in a separate pull request (and it requires some more work as well, such as an escodegen upgrade, node version requirement upgrade, and more extensive testing). If you can reduce this pull request again to just the karma notIncluded part, I'm happy to merge and release a new version.
-- Jimi
Sorry, about the sloppy pull request. I didn't understand how github was going to handle this. I'll close this and start a new pull request, hopefully with just the one change.
This is my simplistic way of getting the included flag set to false for certain files. Created to address #60