Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

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

Open
ni-max wants to merge 1 commit into MagicStack:master
base: master
Choose a base branch
Loading
from ni-max:fix-regex-error

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

Levitanus and chris-vecchio reacted with thumbs up emoji
...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.
Copy link

Confirm it works in ST4

ni-max reacted with thumbs up emoji

Copy link

Levitanus commented Nov 24, 2021
edited
Loading

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)

изображение

изображение

chris-vecchio reacted with thumbs up emoji

Copy link
Member

1st1 commented Oct 20, 2022

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

Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /