Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

[fix issue231] Update sys references to util references. #233

Merged
merged 1 commit into from Jun 28, 2012

Conversation

mojit0
Copy link
Contributor

@mojit0 mojit0 commented Jun 26, 2012

Same edits as the automated pull request from GitHub but without all the noisy commit overhead :)

@rwaldura
Copy link
Contributor

Scoot: just curious, do you know why/how all those commits get into the first pull request?

@mojit0
Copy link
Contributor Author

mojit0 commented Jun 26, 2012

Typically has to do with how the pull request's base branch was created. In this case it appears they didn't do a clean checkout and/or rebase sequence prior to their editing/pull request creation.

@isao
Copy link
Contributor

isao commented Jun 26, 2012

👍 "sys" was renamed util in node.js 0.3.0 https://raw.github.com/joyent/node/v0.3.0/ChangeLog so it should not present any problems for ynodejs04

@isao
Copy link
Contributor

isao commented Jun 28, 2012

for some (probably unrelated) reason this fails mojito-github-pull (job 22)

@mojit0
Copy link
Contributor Author

mojit0 commented Jun 28, 2012

The two test failures in that job have been around since at least build #84 (developer guide test bugs).

isao pushed a commit that referenced this pull request Jun 28, 2012
[fix #231] Update sys references to util references.
@isao isao merged commit 45c0a35 into YahooArchive:develop Jun 28, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants