-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add --config option to the ssc init #239
Conversation
I'm also going to implement this one here #51 |
Maybe it's worthwhile just checking if files exist before over writing them? |
I agree with @jwerle here. I think the creation of the directory and files is helpful but should be guarded on whether they don't yet exist, rather than either a potential error OR (worse) overwriting them. |
ac7323b
to
6103874
Compare
@@ -960,7 +960,7 @@ int main (const int argc, const char* argv[]) { | |||
} | |||
|
|||
// default values | |||
settings["build_output"] = settings["build_output"].size() > 0 ? settings["build_output"] : "dist"; | |||
settings["build_output"] = settings["build_output"].size() > 0 ? settings["build_output"] : "build"; |
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.
so now we have "build"
by default in both .gitignore
and as an implicit default value
made it exit early when |
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.
LGTM
d6e5033
to
34b7cc6
Compare
34b7cc6
to
771afdb
Compare
input = "src" | ||
|
||
; The name of the program and executable to be output. Can't contain spaces or special characters. Required field. | ||
name = "beepboop" | ||
|
||
; The binary output path. It's recommended to add this path to .gitignore. | ||
output = "dist" | ||
output = "build" |
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.
finally, thank you
This helps to wrap the existing application into Socket Runtime