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

Change Syntax Highlighting plugin to fast-syntax-highlighting #8

Open
wants to merge 3 commits into
base: um40
Choose a base branch
from

Conversation

VehementHam
Copy link
Contributor

F-Sy-H is a far better alternative to zsh-syntax-highlighting. It has not only better syntax highlighting, but it is mush less buggy.

F-Sy-H is a far better alternative to zsh-syntax-highlighting. It has not only better syntax highlighting, but it is mush less buggy.
@VehementHam
Copy link
Contributor Author

These changes might be better fit for a later verion of Ultramarine.

@VehementHam VehementHam changed the base branch from um39 to um40 March 28, 2024 21:46
@@ -12,7 +12,7 @@ Source3: ultramarine-shell.sh
Source4: starship.toml

Requires: zsh-autosuggestions
Requires: zsh-syntax-highlighting
Requires: F-Sy-H
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think F-Sy-H is packaged in Fedora or Terra, is it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would need to be packaged, then. I don't know how to do that, but I assume that it is easy to do for shell plugins.

@VehementHam
Copy link
Contributor Author

terrapkg/packages#1037

@lleyton
Copy link
Contributor

lleyton commented May 7, 2024

Yeah, I don't feel comfortable including this plugin considering the sketchiness around the author. See: https://www.youtube.com/watch?v=pHVbjLTFmVo

If you have another option, please let me know.

@VehementHam
Copy link
Contributor Author

@VehementHam
Copy link
Contributor Author

zdharma-continuum is the real deal. He has not done anything suspicious before. Even the commit history is retained.

@lleyton
Copy link
Contributor

lleyton commented May 8, 2024

Looks good with me, package it for Terra and we can ship that

@VehementHam
Copy link
Contributor Author

Alright, great.

@VehementHam VehementHam changed the title Change Syntax Highlighting plugin to F-Sy-H Change Syntax Highlighting plugin to fast-syntax-highlighting May 9, 2024
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.

3 participants