-
Notifications
You must be signed in to change notification settings - Fork 56
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
t2 init: broad cleanup #779
Conversation
e1a36f7
to
17c1216
Compare
controller.installDrivers = function() { | ||
return drivers.install(); | ||
}; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just being moved to lib/controller.js
@@ -33,14 +31,14 @@ exportables.loadNpm = () => { | |||
}; | |||
|
|||
// Resolve an npm cofig list, or nothing (existance is not needed) | |||
exportables.getNpmConfig = (npm) => { | |||
exportables.resolveNpmConfig = (npm) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, I think things that are "resolved" by a promise should be named accordingly.
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
} | ||
// Copy over config file, the blinky main, and the toml file | ||
fs.createReadStream(path.resolve(__dirname, './../../resources/rust/', file_toml)).pipe(fs.createWriteStream(path_toml)); | ||
fs.createReadStream(path.resolve(__dirname, './../../resources/rust/', cargoToml)).pipe(fs.createWriteStream(cargoTomlDir)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this be simplified to use fs.copy
or fs.copySync
is we want to keep it synchronous, like in javascript.js
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you're right. Revision to follow
… string Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
log.info(`Wrote "Hello World" to ${dirAndMainRs}`); | ||
resolve(); | ||
}); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Awesome. Thanks for tackling this @rwaldron. Once tests pass LGTM 👍 |
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Thanks @HipsterBrown! |
WIP. I still have more JS tests to write.
Signed-off-by: Rick Waldron waldron.rick@gmail.com