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

mysql-client formula #34

Merged
merged 6 commits into from
Dec 1, 2015
Merged

mysql-client formula #34

merged 6 commits into from
Dec 1, 2015

Conversation

kyle-long
Copy link
Contributor

No description provided.

@fidian
Copy link
Contributor

fidian commented Dec 1, 2015

I would prefer that the server not require the client. Would you be willing to remove the addition of mysql-client in the server's depends file?

Also the README.md in the new formula needs a header. That's one bit of the automatic documentation that's still manual.

For its own README, but also so the formula README displays the
correct thing
@kyle-long
Copy link
Contributor Author

Sure. I think that will do it

fidian added a commit that referenced this pull request Dec 1, 2015
@fidian fidian merged commit 28a90a3 into tests-always-included:master Dec 1, 2015
@fidian
Copy link
Contributor

fidian commented Dec 1, 2015

Looks great! Thanks!

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