search

astoff / devdocs.el

Emacs viewer for DevDocs
279 stars 16 forks source link

Fix #34 Make `devdocs-install` exclude already installed docs #35

Closed stardiviner closed 3 months ago

stardiviner commented 4 months ago

Fix #34

astoff commented 4 months ago

With this change, you can't reinstall / update a installed document...

I think I understand where you want to get with this, see if https://github.com/astoff/devdocs.el/issues/29#issuecomment-1828150320 is relevant. I don't see an easy fix for the problem, though.

stardiviner commented 4 months ago

I don't think so, the word install means doc is not installed, if user want to update doc, then should call command like devdocs-update. WDYT? If a package manager use install to upgrade package, I think any user will be confused.

stardiviner commented 3 months ago

Hi, any update on this PR? I don't think https://github.com/astoff/devdocs.el/issues/29#issuecomment-1828150320 is very same with this PR purpose.

astoff commented 3 months ago

As mentioned above, this change would make it impossible to reinstall an already installed document.