-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
added ziglang support #5429
added ziglang support #5429
Conversation
Hey, thanks a lot for the contribution. 🤗 The build is currently failing with:
If we can take a look into that, we can merge it afterwards. |
Yeah , soon I will fix it , give some time
Yeah I am also not a zig developer 😅, I have written this with the help of |
Hi, could you delete the commented code and delete the |
Done ✅ |
I don't have idea about why |
I also see these errors when opening the mode in the kitchen sink. These are the reported rules that give errors: [
{
"rule": {
"token": "text",
"regex": "(\\\\s*)(?=\\\\s*(?:[a-zA-Z_][\\\\w.]*|@\\\\\\".+\\\\\\")?)(\\\\s*)((?:!)?)(\\\\s*)([a-zA-Z_][\\\\w.]*|@\\\\\\".+\\\\\\")",
"onMatch": null
},
"groupCount": 5
},
{
"rule": {
"token": "text",
"regex": "(\\\\s*)((?:[a-zA-Z_][\\\\w.]*|@\\\\\\".+\\\\\\")?)((?:!)?)(\\\\s*)(?:([a-zA-Z_][\\\\w.]*|@\\\\\\".+\\\\\\")\\\\b(?!\\\\s*\\\\\\\\())?)",
"onMatch": null
},
"groupCount": 6
}
] |
why this error ? due to any wrong regex or anything else? |
The first rule
The second rule
The regex is a bit too complicated for me to read, so I cannot see what would be the correct order of classes. |
@whazor thanks for pointing out the problem! |
@whazor I have fixed the regex issue |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5429 +/- ##
==========================================
- Coverage 87.59% 86.50% -1.09%
==========================================
Files 584 584
Lines 46408 42376 -4032
Branches 7021 7055 +34
==========================================
- Hits 40650 36657 -3993
+ Misses 5758 5719 -39
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@whazor still there is problem? |
LGTM, I think we can ignore code coverage for language modes |
Issue #, if available:
#5428
Description of changes:
added the zig mode and syntax higlighting and also a sample example for kitchen sink
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.