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

Compatibility with znap #317

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

Conversation

cweagans
Copy link

Geometry currently doesn't work with znap. Details here: marlonrichert/zsh-snap#67

This PR fixes that problem without changing any Geometry behaviors + documents how to install with znap.

@jedahan
Copy link
Member

jedahan commented Jan 2, 2025

The functions directory is part of the source code of geometry, it should be read-only. Updating now, and making our sourcing better in use-fpath branch. Is there a minimal test reproduction we can try to make sure this is compatible with znap?

@jedahan
Copy link
Member

jedahan commented Jan 2, 2025

Trying another stab it at with the function-safelist branch

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