Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix / Workaround for: Error in regex: target of repeat operator is invalid in regex #244

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

Conversation

ni-max
Copy link

@ni-max ni-max commented Nov 22, 2021

This is a workaround for: #242

Feel free to reject this as I believe this is an upstream bug.
But just for the convenience, I'll post this PR as a workaround.

SublimeText 4 (as of build 4121) evaluates this valid regex: \n? to this invalid one: $?
hence causing the following error:

error: Error loading syntax file "Packages/<Path_To>.tmLanguage": Error in regex: target of repeat operator is invalid in regex

…valid in regex

Fixes: MagicStack#242

SublimeText 4 (as of build 4121) evaluates this valid regex: `\n?` to this invalid one: `$?`
hence causing the following error:

```
error: Error loading syntax file "Packages/<Path_To>.tmLanguage": Error in regex: target of repeat operator is invalid in regex
```

This commit is a workaround to fix this problem.
@Levitanus
Copy link

Confirm it works in ST4

@Levitanus
Copy link

Levitanus commented Nov 24, 2021

I've also made a custom colorScheme for MagicPython inspired by both: Solorized and Monokai.

But cannot assume the best way to distribute it: PackageControl is too heavy and long to publish. Just GitHub repo — nobody finds it)

изображение

изображение

@1st1
Copy link
Member

1st1 commented Oct 20, 2022

Is this still the case or can we close this PR?

@Levitanus
Copy link

I think, that still use this branch. But I'm not sure, too long I have not looked into my ST config.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants