-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat: display changelog on :LvimUpdate #3270
base: master
Are you sure you want to change the base?
feat: display changelog on :LvimUpdate #3270
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few things to keep in mind
- treating
callback
ason_done
isn't reliable/consistent depending on what packer is doing - this shouldn't trigger in headless mode
@@ -68,14 +69,15 @@ function M.run_post_update() | |||
M.reset_cache() | |||
|
|||
Log:debug "Syncing core plugins" | |||
require("lvim.plugin-loader").sync_core_plugins() | |||
require("lvim.plugin-loader").sync_core_plugins(function() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the wrong function, you're probably thinking of update_base_lvim()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no I put it there on purpose, because putting make_changelog
in update_base_lvim()
or in line 81, displays the window before packer's window and when packer window kicks in it overlaps with changelog window, so showing it after packer is done makes more sense
|
||
if not in_headless then | ||
vim.schedule(function() | ||
if package.loaded["nvim-treesitter"] then | ||
vim.cmd [[ TSUpdateSync ]] | ||
end | ||
-- TODO: add a changelog | ||
vim.notify("Update complete", vim.log.levels.INFO) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vim.notify("Update complete", vim.log.levels.INFO) | |
Info.make_changelog() | |
vim.notify("Update complete", vim.log.levels.INFO) |
instead of #3218
Everytime I run
:LvimUpdate
I have to go to github and check the commits log on LunarVim repo, so I thought it would be nice if lvim displays them after running:LvimUpdate
, the PR still WIP, waiting for your thoughts on this