Skip to content

Improved nvim checks #448

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improved nvim checks #448

wants to merge 2 commits into from

Conversation

Kasui92
Copy link
Contributor

@Kasui92 Kasui92 commented May 5, 2025

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.

@Kasui92
Copy link
Contributor Author

Kasui92 commented May 19, 2025

@dhh

Schermata 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants