zdharma-continuum / zinit-annex-bin-gem-node

Unmaintained mirror of zinit-zsh/z-a-bin-gem-node
Other
7 stars 9 forks source link

maint: switch from `print -P %F{col-nr} to m {col-name} #28

Open psprint opened 1 year ago

psprint commented 1 year ago

The m function is automatically provided for the moment of loading/installing a plugin and it's the same as +zinit-message.

Description

Allow theming of bin-gem-node messages.

Motivation and Context

There is a problem because the color codes are currently hard-coded into bin-gem-node. With this patch, the messages will make use of +zinit-message theming.

How Has This Been Tested?

Types of changes

Checklist:

psprint commented 1 year ago

@vladdoster @pschmitt @alichtman ping

vladdoster commented 1 year ago

Please use +zi-log instead of m. m gives developers no information about what it does.

psprint commented 1 year ago

Please use +zi-log instead of m. m gives developers no information about what it does.

@vladdoster I've switched to +zi-log. That said, it would be good to document m function. It's designed to be used in atclone/atpull ices, to maximally shorten the code entered into them.