search

romkatv / zsh-bench

Benchmark for interactive Zsh
MIT License
624 stars 27 forks source link

Add configs/ahmubashshir #10

Closed ahmubashshir closed 2 years ago

ahmubashshir commented 2 years ago

Signed-off-by: Mubashshir ahmubashshir@gmail.com

romkatv commented 2 years ago

What's the motivation for adding this config to zsh-bench?

ahmubashshir commented 2 years ago

Some extreme config splitting, maybe!? Mubashshir

On Thu, Dec 2 2021 at 01:57:43 AM -0800, Roman Perepelitsa @.***> wrote:

What's the motivation for adding this config to zsh-bench?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/romkatv/zsh-bench/pull/10#issuecomment-984470554, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5WM3SSER7EWRZ57KVMDBDUO47JPANCNFSM5JGVMBBQ.

romkatv commented 2 years ago

What do you mean by that?

ahmubashshir commented 2 years ago

I tried to split tasks(env initializations/plugin loading/...) into separate files, and loaded them using zsh-defer 😛 Mubashshir

On Thu, Dec 2 2021 at 02:02:13 AM -0800, Roman Perepelitsa @.***> wrote:

What do you mean by that?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/romkatv/zsh-bench/pull/10#issuecomment-984474074, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5WM3UEDFYIEVUGXS3OWADUO472LANCNFSM5JGVMBBQ.

romkatv commented 2 years ago

Cool I guess.

Did you send this PR with a suggestion that I merge it? If so, why do you think I should merge it? In other words, how does this PR improve zsh-bench?

ahmubashshir commented 2 years ago

So that someone won't follow my footstep tweaking so much and instead use zsh4humans, if he wants to tweak even after getting warned by this, this can be an example :3 Mubashshir

On Thu, Dec 2 2021 at 02:07:59 AM -0800, Roman Perepelitsa @.***> wrote:

Cool I guess.

Did you send this PR with a suggestion that I merge it? If so, why do you think I should merge it? In other words, how does this PR improve zsh-bench?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/romkatv/zsh-bench/pull/10#issuecomment-984479386, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5WM3SGZIYPPCA5EXHXYLLUO5AP7ANCNFSM5JGVMBBQ.

romkatv commented 2 years ago

Are you suggesting that I merge this as an example of a complex and bad config? This doesn't seem worthwhile. There is virtually unlimited supply of complex and bad configs.

ahmubashshir commented 2 years ago

ok, you can close it then 😛 Mubashshir

On Thu, Dec 2 2021 at 02:14:49 AM -0800, Roman Perepelitsa @.***> wrote:

Are you suggesting that I merge this as an example of a complex and bad config? This doesn't seem worthwhile. There is virtually unlimited supply of complex and bad configs.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/romkatv/zsh-bench/pull/10#issuecomment-984484819, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5WM3SM5S2XQJXZJMTADDLUO5BJTANCNFSM5JGVMBBQ.

romkatv commented 2 years ago

In the future when you send a PR, provide an explanation why you think the PR is an improvement for the project. It'll save time.