-
Notifications
You must be signed in to change notification settings - Fork 902
Add Chapel to the list of supported language #518
base: master
Are you sure you want to change the base?
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
@googlebot I signed it! |
CLAs look good, thanks! |
src/lang-chapel.js
Outdated
|
||
[ | ||
[PR['PR_KEYWORD'], | ||
/^(?:align|as|atomic|begin|break|by|catch|class|cobegin|coforall|continue|delete|dmapped|do|else|enum|except|export|extern|for|forall|if|index|inline|in|iter|label|lambda|let|local|module|new|noinit|on|only|otherwise|pragma|private|proc|prototype|public|record|reduce|require|return|scan|select|serial|then|throw|throws|try|union|use|var|when|where|while|with|yield|zip|bool|real|int|uint|imag|complex|dmap|domain|string|range|tuple|config|const|param|private|public|sparse|single|type|enum)\b/, null], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you've missed keyword 'sync' here.
Thanks @aditmehta9 I have added it . Please suggest if any other improvement is necessary |
Guys, |
@AliabbasMerchant, @victor-ludorum: I opened issue #587 in hopes of determining what needs to happen next to move your PR forward. (@AliabbasMerchant, I assume your question is directed toward the code-prettify core developers and not at Victor or myself?) |
@bradcray Sir, I have made pull request kindly review it