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.

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

remove support for title method in Casks #7825

Merged
merged 1 commit into from
Dec 6, 2014

Conversation

rolandwalker
Copy link
Contributor

obsoleted by token

HOLD until #7820 and Homebrew/homebrew-cask-versions#589

@rolandwalker rolandwalker added the core Issue with Homebrew itself rather than with a specific cask. label Dec 5, 2014
@claui
Copy link
Contributor

claui commented Dec 5, 2014

Do we have this one already covered in some branch?

@rolandwalker
Copy link
Contributor Author

No, missed that — thanks. @phinze wants us to switch everything to RSpec, and he got us started, but I haven't learned anything about it yet.

@claui
Copy link
Contributor

claui commented Dec 5, 2014

Just did a full-text search which yielded just one more occurrence of title.

Other than that, and apart from the two PRs on whom this one depends, LGTM 👍

@rolandwalker
Copy link
Contributor Author

And that's why I need review!

@rolandwalker rolandwalker changed the title HOLD: remove support for title method in Casks remove support for title method in Casks Dec 6, 2014
rolandwalker added a commit that referenced this pull request Dec 6, 2014
remove support for `title` method in Casks
@rolandwalker rolandwalker merged commit 27c49e3 into Homebrew:master Dec 6, 2014
@rolandwalker rolandwalker deleted the remove_title_support branch December 6, 2014 12:56
@miccal miccal removed the core Issue with Homebrew itself rather than with a specific cask. label Dec 23, 2016
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants