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

make brew and fpm config more alike #202

Merged
merged 2 commits into from
Apr 21, 2017
Merged

make brew and fpm config more alike #202

merged 2 commits into from
Apr 21, 2017

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Apr 21, 2017

brew was discovering homepage and description from the github repository info, which was not documented at all and there were no way of overriding that. So, I added those settings, and also remove the options from fpm section, they all lay in the root now.
Also renamed fpm.url to fpm.homepage.

@bep this may impact you, I'm not doing it in a backwards compatible way because probably no one is using it yet :P

@mention-bot
Copy link

@caarlos0, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jorinvo, @morpheu and @lucapette to be potential reviewers.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.9%) to 91.751% when pulling 5b3484b on brew-fpm-alike into 26d71b3 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.9%) to 91.751% when pulling f2b82ab on brew-fpm-alike into 26d71b3 on master.

@caarlos0 caarlos0 merged commit 8d914d1 into master Apr 21, 2017
@caarlos0 caarlos0 deleted the brew-fpm-alike branch April 21, 2017 20:18
@github-actions
Copy link
Contributor

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 19, 2020
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.

None yet

3 participants