8000 Improved nvim checks by Kasui92 · Pull Request #448 · omacom/omakub · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Aug 30, 2026. It is now read-only.

Improved nvim checks - #448

Merged
dhh merged 2 commits into
omacom:masterfrom
omakasui:nvim-update
Sep 11, 2025
Merged

dhh merged 2 commits into
omacom:masterfrom
omakasui:nvim-update

Conversation

@Kasui92
@Kasui92 Kasui92 commented May 5, 2025 •
Copy link
Copy Markdown
Contributor

I noticed that when I update nvim from Omakub TUI it still regenerates the desktop app from the package.
I think it's okay to remove it.

Also, update nvim theme only if it's actually installed.

This PR implements #443 and cleans the git history from rebase.

Comment thread install/terminal/app-neovim.sh Resolved
@Kasui92
Kasui92 commented May 19, 2025 •
Copy link
Copy Markdown
Contributor Author

@dhh

Sch
8000
ermata da 2025-05-19 22-32-24

In the screenshot above, tested on a fresh install of Omakub on Ubuntu 25.04, I ran the Neovim update and this happened to me. I then cleaned everything and removed Neovim from uninstaller and reinstalled again and the situation returned to me as before. In another installation, however, it did not happen to me.

Here is a screenshot with the generated file.

Schermata da 2025-05-19 22-43-59

I think it could be useful to remove from there too just to be safe.

This might also fix #347

@Kasui92 Kasui92 mentioned this pull request Sep 4, 2025
5 tasks done
@dhh
dhh merged commit 28b959b into omacom:master Sep 11, 2025
@Kasui92
Kasui92 deleted the nvim-update branch September 11, 2025 19:07
burno-lima pushed a commit to burno-lima/omakdot that referenced this pull request May 15, 2026
* fix: remove nvim desktop after update

* fix: update nvim theme only if it s installed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0