Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add TotalTerminal Cask #448

Closed
wants to merge 1 commit into from
Closed

Add TotalTerminal Cask #448

wants to merge 1 commit into from

Conversation

fanquake
Copy link
Contributor

@fanquake fanquake commented Jun 1, 2013

No description provided.

@vitorgalvao
Copy link
Member

When you have a class line with uppercase letters after the initial one, like TotalTerminal, you have to separate the filename with hyphen-minuses at each new uppercase letter, i.e. total-terminal.rb, or it won’t work.

@fanquake
Copy link
Contributor Author

fanquake commented Jun 3, 2013

@vitorgalvao renamed and squashed into a single commit. Guessing this will be merged once the uninstall command is complete?

@vitorgalvao
Copy link
Member

Yes, that is correct. If it were merged now, it’d output errors on account of the unrecognised uninstall parameter.

@0xdevalias
Copy link
Contributor

Is uninstall implemented now? (looks like it in the brew cask menu?)

@vitorgalvao
Copy link
Member

@alias1 please see #395, for details (the latter replies).

@phinze
Copy link
Contributor

phinze commented Oct 21, 2013

got this merged over in #395, hooray! 🌞

@phinze phinze closed this Oct 21, 2013
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fanquake @vitorgalvao @0xdevalias @phinze