caddy icon indicating copy to clipboard operation
caddy copied to clipboard

cmd: remove `{add,remove}-package` and `upgrade` commands

Open mohammed90 opened this issue 8 months ago • 4 comments

First half of implementing the proposal #7010. The plugin code is not pushed yet.

Closes #7010

mohammed90 avatar May 10 '25 16:05 mohammed90

When you publish the plugin, can you document that caddy upgrade does not upgrade plugins? (unless I'm wrong about that)

ItsHarper avatar Jul 21 '25 01:07 ItsHarper

When you publish the plugin, can you document that caddy upgrade does not upgrade plugins? (unless I'm wrong about that)

It will upgrade the plugins. Why wouldn't it? It's the same code from Caddy, just ripped out. If my messaging led the confusion anywhere, point it and I'll correct it.

mohammed90 avatar Jul 21 '25 01:07 mohammed90

Looking at the code and doing a test where I built Caddy with an outdated plugin and then ran the upgrade command led me to believe that the upgrade command only updates Caddy, but keeps the same plugin versions.

ItsHarper avatar Jul 22 '25 07:07 ItsHarper

Looking at the code and doing a test where I built Caddy with an outdated plugin and then ran the upgrade command led me to believe that the upgrade command only updates Caddy, but keeps the same plugin versions.

You're right. Sorry, I was confused and misremembered how the code works. Well, more reasons to not have this "feature" builtin 😀

mohammed90 avatar Aug 06 '25 00:08 mohammed90