Skip to content

test: loosen infinite loop detect condition - #1681

Closed
yosuke-furukawa wants to merge 1 commit into
nodejs:masterfrom
yosuke-furukawa:hotfix/test-dgram-send-callback-recursive
Closed

yosuke-furukawa wants to merge 1 commit into
nodejs:masterfrom
yosuke-furukawa:hotfix/test-dgram-send-callback-recursive

Conversation

@yosuke-furukawa

Copy link
Copy Markdown
Member

related #1675

@brendanashworth brendanashworth added test Issues and PRs related to Node.js core tests and test infrastructure. dgram Issues and PRs related to UDP and the dgram module. labels May 12, 2015
@yosuke-furukawa

Copy link
Copy Markdown
Member Author

jenkins ci: https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/663/

I don't have a permission to run jenkins ci. How do I get the permission??

@bnoordhuis

Copy link
Copy Markdown
Member

@rvagg Can you give @yosuke-furukawa access?

@yosuke-furukawa When you have an account, go here, fill in the 'user' and 'branch' fields ('yosuke-furukawa' and 'hotfix/test-dgram-send-callback-recursive' in this case), leave the other fields at their defaults, click 'Build' and go get a cup of coffee - you'll have plenty of time to finish it.

@yosuke-furukawa yosuke-furukawa changed the title test: loosen infinite detection condition test: loosen infinite loop detect condition May 12, 2015
@Fishrock123

Copy link
Copy Markdown
Contributor

@Fishrock123

Copy link
Copy Markdown
Contributor

CI seems happier. The windows failures are unrelated afaik, I'll try to root them out today.

Fishrock123 pushed a commit that referenced this pull request May 14, 2015
Fixes: #1675
PR-URL: #1681
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@Fishrock123

Copy link
Copy Markdown
Contributor

Landed in 4e2f999 thanks!

Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request May 19, 2015
Fixes: nodejs#1675
PR-URL: nodejs#1681
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dgram Issues and PRs related to UDP and the dgram module. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants