Skip to content
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

Updates for Zim 1.x #192

Merged
merged 2 commits into from
Nov 12, 2021
Merged

Updates for Zim 1.x #192

merged 2 commits into from
Nov 12, 2021

Conversation

ericbn
Copy link
Contributor

@ericbn ericbn commented Nov 11, 2021

Hola! 👋 Qué surpresa tan buena encontrar que a ustedes les gusta Zim y lo utilizan en su dotfiles y su curso de Zsh!


See changelog in https://github.com/zimfw/zimfw/blob/master/CHANGELOG.md

Thanks a lot for chosing Zim as the Zsh framework of choice for your dotfiles and your courses!

I spotted a few points where the usage is not compatible with the changes since Zim 1.0.0 (released 2020-01-07):

  • The zimfw repo is not intended to be cloned for daily use. You only need the zimfw.zsh file. And downloading it from https://github.com/zimfw/zimfw/releases/latest/download/zimfw.zsh will always give you the latest released version of it.
  • .zimrc should only contain module definitions, and it's not sourced anymore when the shell starts.
  • zhighlighters is deprecated in favor of ZSH_HIGHLIGHT_HIGHLIGHTERS, since we're using the zsh-users/zsh-syntax-highlighting module directly now.
  • There are two separate zimfw actions: upgrade will upgrade the zimfw.zsh file itself, and update will update the modules. You usually would want to do both if you want to "up" everything...

See changelog in https://github.com/zimfw/zimfw/blob/master/CHANGELOG.md

Thanks a lot for chosing Zim as the Zsh framework of choice for your
dotfiles and your courses!

I spotted a few points where the usage is not compatible with the
changes since Zim 1.0.0 (released 2020-01-07):
* The zimfw repo is not intended to be cloned for daily use. You only
  need the zimfw.zsh file. And downloading it from
  https://github.com/zimfw/zimfw/releases/latest/download/zimfw.zsh
  will always give you the latest released version of it.
* .zimrc should only contain module definitions, and it's not sourced
  anymore when the shell starts.
* zhighlighters is deprecated in favor of ZSH_HIGHLIGHT_HIGHLIGHTERS,
  since we're using the zsh-users/zsh-syntax-highlighting module
  directly now.
* There are two separate zimfw actions: `upgrade` will upgrade the
  zimfw.zsh file itself, and `update` will update the modules.
  You usually would want to do both if you want to "up" everything...
Copy link
Contributor Author

@ericbn ericbn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments more:


zhighlighters=(main brackets)
zmodule zsh-users/zsh-syntax-highlighting
zmodule zsh-users/zsh-autosuggestions
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's recommended to source zsh-users/zsh-syntax-highlighting after completion. Likewise, zsh-users/zsh-autosuggestions goes better after zsh-users/zsh-syntax-highlighting.

dotfiles_template/shell/zsh/.zshenv Outdated Show resolved Hide resolved
find "$DOTLY_PATH/modules/zimfw" -name '*.old' -exec rm -rf {} \;

/bin/zsh -c "source ${ZDOTDIR:-${HOME}}/.zlogin"
zsh "$ZIM_HOME/zimfw.zsh" clean-compiled && zsh "$ZIM_HOME/zimfw.zsh" compile
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two zimfw actions will do what you were doing before.

Copy link
Member

@rgomezcasas rgomezcasas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks you for creating Zim! 😊

I've modified the path where zim is installed. Also, thanks for all the advices. 😊

dotfiles_template/shell/zsh/.zshenv Outdated Show resolved Hide resolved
scripts/self/install Outdated Show resolved Hide resolved
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