qunitjs / js-reporters

📋 Common Reporter Interface (CRI) for JavaScript testing frameworks.
MIT License
60 stars 18 forks source link

Import Basic Structure #13

Closed fcarstens closed 9 years ago

fcarstens commented 9 years ago

I haven't included the other adapters yet. If the structure is okay this would be my next step. Diff view without QUnit: https://github.com/js-reporters/js-reporters/commit/65d1b17fe3be9568bd2bb51f50f24b766a81c465

jzaefferer commented 9 years ago

Let's add some linting to this repo, using jshint+jscs or eslint. Preferably some preset that doesn't require any custom repo-specific rules.

fcarstens commented 9 years ago

I fixed all other issues and will take a look at linting tomorrow. :smiley:

fcarstens commented 9 years ago

Resolved all issues mentioned above except for runStart. Will work on that after finishing the Jasmine Adapter.

fcarstens commented 9 years ago

mkdir -p doesn't work on windows. I included an node package which handles this. :smiley:

jzaefferer commented 9 years ago

mkdirp looks good. Some whitespace linting and this is good to land.

fcarstens commented 9 years ago

Done! :smiley:

jzaefferer commented 9 years ago

Looks good! Rebased into a single commit. Looking forward to the JasmineAdapter.