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

zsh: fix HISTSIZE configuration #178

Closed
wants to merge 2 commits into from

Conversation

uvNikita
Copy link
Collaborator

HISTSIZE has to be set before sourcing oh-my-zsh
since otherwise it will be overridden. Fixes #177.

@rycee
Copy link
Member

rycee commented Jan 12, 2018

I'm no expert but this looks fine to me. The only thing I can comment on is that the cfg. in zsh: add cfg.history.save option seems unnecessary, I think it is fair to assume that any option mentioned in a commit message for a specific module will be relative the module "option base path".

Thanks for keeping on top of the zsh module! :-)

HISTSIZE and HISTFILE should be set in ~/.zshrc and before
sourcing oh-my-zsh since otherwise it will be overridden.
Fixes nix-community#177.
@uvNikita
Copy link
Collaborator Author

Agree with the comment on the commit message, it was due to a copy-paste, so thanks for pointing it out. :)
Rebased into master: a93445f

@uvNikita uvNikita closed this Jan 13, 2018
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.

None yet

2 participants