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

[Tests] add test case for #519 for test.comment() in createStream/objectMode context #520

Merged
merged 1 commit into from
May 25, 2020

Conversation

DavidAnson
Copy link
Contributor

No description provided.

@ljharb
Copy link
Collaborator

ljharb commented May 25, 2020

oops! this wasn't linked to #519, so i committed fb94836 before realizing this was here.

Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Keeping this for the extra test case. Thanks!

@ljharb ljharb changed the title Fix test.comment() when used in createStream/objectMode context (fixes #519). [Tests] add test case for #519 for test.comment() in createStream/objectMode context May 25, 2020
@ljharb ljharb merged commit 40ec79a into tape-testing:master May 25, 2020
@DavidAnson
Copy link
Contributor Author

OK, thanks! Out of curiosity, how should I have linked it to the issue?

@ljharb
Copy link
Collaborator

ljharb commented May 25, 2020

Just referencing it with # and the number, like I did in my comment, would be sufficient :-) You can also comment on the issue with a similar reference to the PR number.

@DavidAnson
Copy link
Contributor Author

I put “fixes ###” in the commit message and it did show up in the issue timeline. No worries here, just wanted to be sure I was doing things right.

@ljharb
Copy link
Collaborator

ljharb commented May 25, 2020

ah, you're right! however, a commit reference in the timeline is unfortunately a lot more subtle than a PR reference, so i missed it :-/

@DavidAnson
Copy link
Contributor Author

Got it. I’ll be more explicit in the future!

ljharb added a commit that referenced this pull request Dec 29, 2020
 - [New] Include name of test in log when test times out (#524)
 - [readme] document Promise support; remove Promise-related alternatives
 - [readme] add `tape-describe` to 'other' section (#523)
 - [Deps] update `deep-equal`, `is-regex`, `object-inspect`, `object-is`, `object.assign`, `resolve`, `string.prototype.trim`
 - [Dev Deps] update `eslint`, `js-yaml`
 - [eslint] remove useless regex escapes
 - [Tests] handle stack differences in node 15
 - [Tests] add test case for #519 for test.comment() in createStream/objectMode context (#520)
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