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

removed unused fs module #217

Merged
merged 1 commit into from
Jul 5, 2014
Merged

removed unused fs module #217

merged 1 commit into from
Jul 5, 2014

Conversation

tonylukasavage
Copy link
Contributor

The fs module and a reference to fs.existsSync are included in the variable declarations. They are unused, so I removed them.

@Mithgol
Copy link
Contributor

Mithgol commented May 7, 2014

I've noticed “The Travis CI build could not complete due to an error” message and opened a new pull request (#218) about it.

@tonylukasavage
Copy link
Contributor Author

Thanks @Mithgol for getting that resolved.

@SomeKittens
Copy link
Collaborator

This appears to have been resolved.

@SomeKittens SomeKittens closed this Jul 4, 2014
@zhiyelee zhiyelee reopened this Jul 5, 2014
@zhiyelee
Copy link
Collaborator

zhiyelee commented Jul 5, 2014

hi @SomeKittens, @Mithgol fixed travis config in #218, but this pr still work.

zhiyelee added a commit that referenced this pull request Jul 5, 2014
@zhiyelee zhiyelee merged commit ebdd789 into tj:master Jul 5, 2014
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.

4 participants