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

Invalid regular expression: invalid group specifier name #1687

Closed
ytyt-yt opened this issue Oct 6, 2020 · 5 comments · Fixed by #1689
Closed

Invalid regular expression: invalid group specifier name #1687

ytyt-yt opened this issue Oct 6, 2020 · 5 comments · Fixed by #1689

Comments

@ytyt-yt
Copy link

ytyt-yt commented Oct 6, 2020

lookbehind assertions is not supported by safari.

This line will crash the app on safari.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Browser_compatibility

@me-momo

@hustcc
Copy link
Member

hustcc commented Oct 8, 2020

@ytyt-yt refactor the function, pr is here #1689 , help give us a code review.

@dreamkai
Copy link

dreamkai commented Oct 8, 2020

问题在于正式环境部署,jenkins自动构建,还是会安装依赖,如何解决

@hustcc
Copy link
Member

hustcc commented Oct 8, 2020

问题在于正式环境部署,jenkins自动构建,还是会安装依赖,如何解决

这样,你把依赖改成 github 地址,这样保证代码最新,然后在你们环境验证 ok 了之后,我们就发布一个版本。

@dreamkai
Copy link

dreamkai commented Oct 8, 2020 via email

@hustcc
Copy link
Member

hustcc commented Oct 9, 2020

v2.0.2 released.

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 a pull request may close this issue.

3 participants