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

Update travis badge. #142

Merged
merged 1 commit into from Nov 2, 2017
Merged

Update travis badge. #142

merged 1 commit into from Nov 2, 2017

Conversation

priyank-p
Copy link
Contributor

No description provided.

cjihrig

This comment was marked as off-topic.

@joyeecheung
Copy link
Member

joyeecheung commented Oct 30, 2017

On a side note, what's the status with the Travis CI? At the moment all builds fail with test timed out after 15000ms, looks like lldb printed some errors on the test commands, has somebody looked into that? I can investigate that if no one is working on it...or are we moving to Jenkins? (nodejs/build#777)

@priyank-p
Copy link
Contributor Author

@joyeecheung tape test/*-test.js command is timeout for some reason.

@joyeecheung
Copy link
Member

@cPhost Yes it times out when the expected output does not come out from the lldb session, most of the time it's because lldb got into some errors so instead of the expected output, it writes the error message. We should probably improve the tests by setting a timeout ourselves and log the session output when it times out.

joyeecheung

This comment was marked as off-topic.

@joyeecheung joyeecheung merged commit efaf023 into nodejs:master Nov 2, 2017
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.

None yet

3 participants