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

Skip zwc zsh function files #743

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

bengolder
Copy link

@bengolder bengolder commented Apr 25, 2024

  • Uses the ~ glob operator to be consistent with `_load_settings()``
  • moved loading after _load_settings, because setopt extendedglob is needed for the ~ glob operator and is run in _load_settings()

- Uses the `~` glob operator to be consistent with `_load_settings()``
- moved loading after `_load_settings``, because `setopt extendedglob`
  is needed for the `~` glob operator and is run in `_load_settings()`
@bengolder
Copy link
Author

Closes #742

@bengolder
Copy link
Author

It seemed harmless and potentially only beneficial to move the loading of zsh functions to after _load_settings. I figured if anything, it allows the zsh functions to take advantage of any zsh configs, including the setopt extendedglob.

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.

1 participant