Skip to content

Setup initial command skeleton #7

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

psyGamer
Copy link
Member

No description provided.

Copy link
Member

@microlith57 microlith57 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a couple wording changes but otherwise looks good!

System.Console.WriteLine($"TODO: Install Everest: '{name}' '{everestVersion}'");
}

[Command("uninstall", Description = "Uninstalls Everest from an installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("uninstall", Description = "Uninstalls Everest from an installation")]
[Command("uninstall", Description = "Uninstall Everest from an installation")]

}

public void Run()
[Command("launch", Description = "Launches an installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("launch", Description = "Launches an installation")]
[Command("launch", Description = "Launch an installation")]

});
}

[Command("add", Description = "Add an existing installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("add", Description = "Add an existing installation")]
[Command("add", Description = "Add an existing installation to the list Tholus knows about")]

System.Console.WriteLine($"TODO: Add new install: '{name}' '{path}'");
}

[Command("remove", Description = "Remove an installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("remove", Description = "Remove an installation")]
[Command("remove", Description = "Remove an installation from the list Tholus knows about")]

System.Console.WriteLine($"TODO: Create mod: '{install}' '{template}' '{mod}'");
}

[Command("build", Description = "Builds a mod's code for an installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("build", Description = "Builds a mod's code for an installation")]
[Command("build", Description = "Build a mod's source code")]

System.Console.WriteLine($"TODO: Add new mod to bag: '{bag}' '{mod}'");
}

[Command("remove", Description = "Removes a mod from the mod bag")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("remove", Description = "Removes a mod from the mod bag")]
[Command("remove", Description = "Remove a mod from a bag")]

System.Console.WriteLine($"TODO: Remove mod from bag: '{bag}' '{mod}'");
}

[Command("list", Description = "Lists all mods of the mod bag")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("list", Description = "Lists all mods of the mod bag")]
[Command("list", Description = "List the mods contained in a bag")]

System.Console.WriteLine($"TODO: List mods of bag: '{bag}'");
}

[Command("apply", Description = "Applies the mod bag to an installation")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("apply", Description = "Applies the mod bag to an installation")]
[Command("apply", Description = "Apply a mod bag to an installation")]

System.Console.WriteLine($"TODO: Apply bag: '{bag}' '{install}'");
}

[Command("create", Description = "Creates a new empty mod bag")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("create", Description = "Creates a new empty mod bag")]
[Command("create", Description = "Create a new empty mod bag")]

System.Console.WriteLine($"TODO: Create bag: '{bag}'");
}

[Command("destroy", Description = "Destroies an existing mod bag")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Command("destroy", Description = "Destroies an existing mod bag")]
[Command("destroy", Description = "Destroy an existing mod bag")]

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.

2 participants