Fixing some cross-platform path issues #43

Merged
TehShrike merged 2 commits from master into master 2012-11-30 22:34:20 -08:00
TehShrike commented 2012-11-16 19:16:14 -08:00 (Migrated from github.com)

Mostly just replacing a lot of path + "/" + otherPath lines with _path.join(path, otherPath)

Also removed a few warnings from tests: "path.existsSync is now called fs.existsSync."

Mostly just replacing a lot of `path + "/" + otherPath` lines with `_path.join(path, otherPath)` Also removed a few warnings from tests: "path.existsSync is now called `fs.existsSync`."
TehShrike commented 2012-11-29 16:43:34 -08:00 (Migrated from github.com)

I'm rolling out some modules on Windows systems that are going to be relying on wrench, it'd be cool if this could make its way into the version of wrench on npm in the near future!

I'm rolling out some modules on Windows systems that are going to be relying on wrench, it'd be cool if this could make its way into the version of wrench on npm in the near future!
ryanmcgrath commented 2012-11-30 22:34:53 -08:00 (Migrated from github.com)

Ah, you got it! Done.

Ah, you got it! Done.
This repository is archived. You cannot comment on pull requests.
No reviewers
No milestone
No project
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/wrench-js#43
No description provided.