-
Notifications
You must be signed in to change notification settings - Fork 22
TDD section updates #31
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
Conversation
Codecov Report
Merging #31 into master will not change coverage.
The diff coverage isn/a
.
@@ Coverage Diff @@ ## master #31 +/- ## ===================================== Coverage 100% 100% ===================================== Files 1 1 Lines 43 43 ===================================== Hits 43 43
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d17b772...f6f2204. Read the comment docs.
Adds syntax highlighting, fixes the way the mount function is called (straight arguments instead of an options object), removes a console.log and clears the test DOM dwyl#30
e646831
to
f6f2204
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@finnhodgkin thanks very much for making these updates/clarifications. 🎉
Adds a few tweaks to the TDD sections of the readme and to the example test file.
fixes #30