Skip to content

QUnit.todo Support#1400

Merged
step2yeung merged 5 commits intotestem:masterfrom
wagenet:todo-support
Mar 10, 2020
Merged

QUnit.todo Support#1400
step2yeung merged 5 commits intotestem:masterfrom
wagenet:todo-support

Conversation

@wagenet
Copy link
Contributor

@wagenet wagenet commented Feb 14, 2020

Currently includes #1398 and #1399. Should be rebased once those are merged.


Supersedes #1101

Includes support for TAP, Dot, and dev reporters. May not work properly with other options.

@wagenet wagenet requested a review from step2yeung February 14, 2020 20:56
@wagenet wagenet force-pushed the todo-support branch 2 times, most recently from af9020c to 339074d Compare February 14, 2020 21:50
@wagenet
Copy link
Contributor Author

wagenet commented Feb 14, 2020

I've tested this against an Ember app using QUnit.todo and it works well enough for me to want to use it. Of course, I understand if there's a higher bar for merging :)

Copy link
Member

@step2yeung step2yeung left a comment

Choose a reason for hiding this comment

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

Thanks for progressing on #1101 @wagenet !!
Two comments left, I like the strict option you added!

@wagenet
Copy link
Contributor Author

wagenet commented Mar 3, 2020

@step2yeung any thoughts?

@step2yeung
Copy link
Member

@wagenet thanks for the ping & your patiences! Resolved 2 of the comments, also just pulling in the pr myself to see how it works :)

@wagenet
Copy link
Contributor Author

wagenet commented Mar 9, 2020

@step2yeung I believe the remaining failures are not my fault.

@step2yeung step2yeung merged commit 091b094 into testem:master Mar 10, 2020
@step2yeung
Copy link
Member

@wagenet merged! Thanks for wrapping up the PR!

fyi @rwjblue

@wagenet wagenet deleted the todo-support branch March 11, 2020 16:31
@rwjblue
Copy link
Contributor

rwjblue commented Mar 15, 2020

Awesome, thank you both!

@step2yeung - Mind shipping a release?

@step2yeung
Copy link
Member

@rwjblue @wagenet released as v3.0.3 !

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