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

Add support for MQL4 and MQL5 #2

Closed
kenorb opened this issue Apr 19, 2018 · 2 comments
Closed

Add support for MQL4 and MQL5 #2

kenorb opened this issue Apr 19, 2018 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed question Further information is requested

Comments

@kenorb
Copy link

kenorb commented Apr 19, 2018

  • MQL4: .mq4
  • MQL5: .mq5
  • MQL Header files: .mqh

Currently it's detected as a plain text.

Related:

@boyter
Copy link
Owner

boyter commented Apr 19, 2018

Would you be able to fill in the rest of the below JSON structure and then I can add it. You will need to create 3, one for MQL4, MQL5 and MQL Header Files.

In here is fine, unless you want to submit a pull request and get the glory :)

"MQL4": {
    "line_comment": [
    ],
    "complexitychecks": [
    ],
    "extensions": [
      "mq4"
    ],
    "multi_line": [],
    "quotes": [
    ]
  },

For reference here is simplified Java one. You can find more examples here https://github.com/boyter/scc/blob/master/languages.json

"Java": {
  "line_comment": [
    "//"
  ],
  "complexitychecks": [
    "for ",
    "for(",
  ],
  "extensions": [
    "java"
  ],
  "multi_line": [
    [
      "/*",
      "*/"
    ]
  ],
  "quotes": [
    [
      "\"",
      "\""
    ]
  ]
}

@boyter boyter added question Further information is requested enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Apr 19, 2018
@boyter
Copy link
Owner

boyter commented Jun 8, 2018

Had a look. Looked easy enough to just copy C++

Added in #13

@boyter boyter closed this as completed Jun 8, 2018
boyter pushed a commit that referenced this issue Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants