search

tomsowerby / starred-package-installer

Starred Package Installer package for Atom.io
MIT License
6 stars 7 forks source link

Check for installed packages #1

Open m1ga opened 10 years ago

m1ga commented 10 years ago

It looks like its not checking if the package is already installed. I've started your package and its installing all my stared packages again

tomsowerby commented 10 years ago

Hi @m1ga Thanks for trying it out, and you are correct unfortunately. I've noticed the same thing, and it would be much quicker if it didn't. This plugin is really just a wrapper to conveiniently run 

apm stars --install

as per this blog post

It would be brilliant if the --install command checked what was there first. The apm source is here if you want to start a PR or issue with them.

Sorry, I feel like this response is a bit blame passing, but it's out of the scope of what I initially created this plugin for (not having to open up the command line).

I can see how it would be valuable though. I'll have a think about the pros and cons of implementing it.

Thanks again.

Tom

m1ga commented 10 years ago

I haven't had a look inside the plugin yet but since you have the same with just the commandline I'll open up a ticket (if there isn't already one) in the apm project.

And yes, I don't want to open the command line too (at least not for that) :) If your project would be included in the main app it would be the first button I would click when I open a fresh install somewhere! Or a button inside the settings.

Thanks for clarifying. I'll include the link here when I create the new report

Edit: already a ticket here: https://github.com/atom/apm/issues/170