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

Fix recursion sanity limit #16

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

keith-hall
Copy link

Avoid with_prototype to fix context recursion sanity limit with latest ShellScript syntax from sublimehq/Packages master branch

michaelblyons added a commit to michaelblyons/SublimeSyntax-ssh_config that referenced this pull request Dec 15, 2024
Incorporates a lot of robballou#16

Co-Authored-By: @keith-hall
@MatthiasPortzel
Copy link

MatthiasPortzel commented Feb 6, 2025

Any progress on this? Makes the syntax unusable for me. Sorry for the cheap "bump" comment.

@michaelblyons
Copy link
Collaborator

Sorry. I've been lazy because it involves some rearranging of PC tagging. I've got a PR for that, now at wbond/package_control_channel#9058 if you want to follow.

I also have a bunch of other improvements built up at #18 if you want to try that.

@michaelblyons
Copy link
Collaborator

FWIW, this PR is also dependent on wbond/package_control_channel#9058. I've merged this into #18 already.

@MatthiasPortzel
Copy link

MatthiasPortzel commented Feb 8, 2025

Pulled #18 locally and it does fix the popup error message. (I can't tell if its actually embedding the syntax highlighting, but it's so minor that it doesn't matter.) Thanks!

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