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

Add test interface option #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xdissent
Copy link

@xdissent xdissent commented Aug 1, 2015

No description provided.

<select class="option-input" id="option-interface">
<option value="object">Object</option>
<option value="tdd">TDD</option>
<option value="bdd">BDD</option>
Copy link

Choose a reason for hiding this comment

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

Probably want to also add the new qunit interface?

Copy link
Author

Choose a reason for hiding this comment

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

I considered it, but I haven't actually used it, and I wasn't sure if the this.remote convention worked within the qunit runner. If it operates the same as the others I'll add it to this PR if you'd like.

@dylans
Copy link

dylans commented Aug 3, 2015

Thanks for the pr. @csnover, I assume there's no technical reason to only support tdd, other than just having the first version have fewer options?

@dylans
Copy link

dylans commented Aug 3, 2015

@xdissent I don't see a CLA on file for you ( please see guidelines at https://github.com/theintern/recorder/blob/master/CONTRIBUTING.md#submitting-pull-requests ).

@xdissent
Copy link
Author

xdissent commented Aug 3, 2015

@dylans Just signed CLA (sorry for the weird erroneous comment from my CI server lol)

@jason0x43 jason0x43 force-pushed the master branch 2 times, most recently from e11de4a to 6741ae3 Compare March 29, 2018 21:56
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.

2 participants