snapcraft plugins -> snapcraft list-plugins #951

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
Contributor

tsdgeos commented Dec 7, 2016

snapcraft plugins does not exist so i guess snapcraft list-plugins is what we really want there?

codecov-io commented Dec 7, 2016

Current coverage is 97.03% (diff: 100%)

Merging #951 into master will not change coverage

@@             master       #951   diff @@
==========================================
  Files           161        161          
  Lines         16367      16367          
  Methods           0          0          
  Messages          0          0          
  Branches       1244       1244          
==========================================
  Hits          15882      15882          
  Misses          300        300          
  Partials        185        185          

Powered by Codecov. Last update a87decf...b3eaaab

elopio approved these changes Dec 11, 2016

We should have snapcraft plugins:
https://bugs.launchpad.net/snapcraft/+bug/1587055

But we don't have it yet. So +1. Thanks Albert!

Collaborator

sergiusens commented Dec 13, 2016

Thanks for looking into this, we should actually support the plugins alias instead (this was written by Product Strategy so we should follow that)

If still interested, all this involves is something like:
https://github.com/snapcore/snapcraft/pull/890/files#diff-07ab19acf782b5f8bd05c71b8d04f963R99

then adding to:
https://github.com/snapcore/snapcraft/pull/890/files#diff-07ab19acf782b5f8bd05c71b8d04f963R256

and a command test in snapcraft/tests/test_commands_list_plugins.py maybe just using testscenarios as in the list-registered one.

This bug exists for this PR -> LP: #1644678

@tsdgeos tsdgeos closed this Dec 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment