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

1c #4132

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

1c #4132

wants to merge 7 commits into from

Conversation

kolesnikovav
Copy link

Issue #, if available:
Added new mode.

Description of changes:

Added 1C Mode support.
1C is the most popular ERP database solution in Russia and ex USSR countries.
Support Russian and English script variant.
Support code folding and highlighting.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Dec 11, 2019

Codecov Report

Attention: Patch coverage is 32.33083% with 90 lines in your changes missing coverage. Please review.

Project coverage is 63.96%. Comparing base (45558c2) to head (aff379f).
Report is 1226 commits behind head on master.

Files with missing lines Patch % Lines
lib/ace/mode/folding/1c.js 15.09% 90 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4132      +/-   ##
==========================================
- Coverage   70.51%   63.96%   -6.55%     
==========================================
  Files         523      444      -79     
  Lines       51157    35785   -15372     
  Branches     9623     6809    -2814     
==========================================
- Hits        36071    22891   -13180     
+ Misses      15086    12894    -2192     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@WinterSilence
Copy link
Contributor

регулярки нужно использовать с модификатором u

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

Successfully merging this pull request may close these issues.

3 participants