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

Add a generic package manager plugin. #466

Merged
merged 4 commits into from Jul 5, 2016
Merged

Add a generic package manager plugin. #466

merged 4 commits into from Jul 5, 2016

Conversation

kdeldycke
Copy link
Contributor

Currently supports Homebrew and Cask.

I plan to add others like Python's pip in future revisions.

@kdeldycke kdeldycke mentioned this pull request Jul 5, 2016
@tresni
Copy link
Collaborator

tresni commented Jul 5, 2016

Please host the image externally (imgur seems to be a popular choice.) This repo only contains scripts, not their screenshots or other ancillary data.

@kdeldycke
Copy link
Contributor Author

@tresni : should I also convert my image to jpeg? Travis tests do not recognise PNG: https://travis-ci.org/matryer/bitbar-plugins/builds/142523996#L346

@tresni
Copy link
Collaborator

tresni commented Jul 5, 2016

No, you need to remove it completely from your PR and host it externally (using your dot-files repo works, but I'd usually recommend an image hosting service such as imgur or imageshack.)

@kdeldycke
Copy link
Contributor Author

@tresni : done. Also fixed a tiny issue about multiline description in metadata.

@tresni
Copy link
Collaborator

tresni commented Jul 5, 2016

👍 Going to start using this myself, nice work :)

@tresni tresni merged commit 1baa8ba into matryer:master Jul 5, 2016
@kdeldycke
Copy link
Contributor Author

Thanks for the merge! That was fast! :)

@kdeldycke kdeldycke deleted the package_manager_plugin branch July 5, 2016 20:42
kdeldycke added a commit to kdeldycke/meta-package-manager that referenced this pull request Aug 17, 2016
* Add a generic package manager plugin.

* Tricks CI tests to workaround chicken and egg issue.

* Host plugin image on imgur.

* Metadata are not allowed on multiple lines.
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.

None yet

2 participants