Skip to content

Add coverage ignores to recorder, and emit text report#1608

Merged
paulmelnikow merged 1 commit intobetafrom
recorder-ignores
Jul 8, 2019
Merged

Add coverage ignores to recorder, and emit text report#1608
paulmelnikow merged 1 commit intobetafrom
recorder-ignores

Conversation

@paulmelnikow
Copy link
Copy Markdown
Member

Ref #1404 (comment)

This leaves intact two missed coverage spots in lib/recorder.js which are being taken care of in #1588.

Also ref #1607

@paulmelnikow paulmelnikow merged commit 2bca61b into beta Jul 8, 2019
@paulmelnikow
Copy link
Copy Markdown
Member Author

Thanks for the reviews!

@paulmelnikow paulmelnikow deleted the recorder-ignores branch July 8, 2019 22:54
@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Jul 9, 2019

🎉 This PR is included in version 11.0.0-beta.22 🎉

The release is available on:

Your semantic-release bot 📦🚀

@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Aug 12, 2019

🎉 This PR is included in version 11.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

gr2m pushed a commit that referenced this pull request Sep 4, 2019
Ref #1404 (comment)

This leaves intact two missed coverage spots in `lib/recorder.js` which are being taken care of in #1588.

Also ref #1607
gr2m pushed a commit that referenced this pull request Sep 4, 2019
Ref #1404 (comment)

This leaves intact two missed coverage spots in `lib/recorder.js` which are being taken care of in #1588.

Also ref #1607
gr2m pushed a commit that referenced this pull request Sep 5, 2019
Ref #1404 (comment)

This leaves intact two missed coverage spots in `lib/recorder.js` which are being taken care of in #1588.

Also ref #1607
juninmd pushed a commit to juninmd/nock that referenced this pull request Mar 21, 2026
Ref nock#1404 (comment)

This leaves intact two missed coverage spots in `lib/recorder.js` which are being taken care of in nock#1588.

Also ref nock#1607
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants