Skip to content

Compatibility with znap#317

Open
cweagans wants to merge 1 commit into
geometry-zsh:mainfrom
cweagans:znap-compat
Open

Compatibility with znap#317
cweagans wants to merge 1 commit into
geometry-zsh:mainfrom
cweagans:znap-compat

Conversation

@cweagans

Copy link
Copy Markdown

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

jedahan commented Jan 2, 2025

Copy link
Copy Markdown
Member

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

jedahan commented Jan 2, 2025

Copy link
Copy Markdown
Member

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