-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
make every instance use a minecraft folder
afde899
to
2ca3e8a
Compare
var argType = Argument("type", | ||
"The type of instance can be 'Vanilla', 'Server' or 'MultiMC'"); | ||
var argName = Argument("name", | ||
"The name of the pack"); | ||
var argDirectory = Argument("[directory]", |
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.
Should the directory argument be removed (and we'll just use the name instead)?
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.
well that would remove some complications
var argDirectory = Argument("[directory]", | ||
"The directory to initialize the pack in. Created if necessary."); | ||
|
||
var optName = Option("-n | --name", | ||
"Sets the pack name", CommandOptionType.SingleValue); | ||
var optDescription = Option("-d | --description", |
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.
I.. guess the description and authors should be removed. Whoever uses init
command will edit the config file manually anyway.
@@ -14,6 +14,10 @@ public interface IInstanceHandler | |||
/// the specified simplified (safe) instance / folder name. </summary> | |||
string GetInstancePath(string instanceName); | |||
|
|||
/// <summary> Gets the full path of an instance for this type with | |||
/// the specified simplified (safe) instance / folder name. </summary> | |||
string GetInstancePath(string instanceName, string basedir); |
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.
Is the old one still used?
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.
well all ues of the old one could be changed by passing in the current working directory
ping #28 for changes in cli arguments and options |
make every instance use a minecraft folder
closes #27