linguist icon indicating copy to clipboard operation
linguist copied to clipboard

Add support for EdgeQL languages

Open DAlperin opened this issue 4 years ago • 3 comments

Implements https://www.edgedb.com/docs/edgeql/index

Description

Adds grammars, language definition, and adds test case.

Checklist:

  • [x] I am adding a new language.

    • [x] The extension of the new language is used in hundreds of repositories on GitHub.com.

      • Search results for each extension:
        • https://github.com/search?q=extension%3Aedgeql+NOT+nothack&type=Code
        • https://cs.github.com/?scopeName=All+repos&scope=&q=path%3A*.edgeql
    • [x] I have included a real-world usage sample for all extensions added in this PR:

      • Sample source(s):
        • https://github.com/edgedb/edgedb-examples
      • Sample license(s):
      • https://github.com/edgedb/edgedb/blob/master/LICENSE

DAlperin avatar Feb 11 '22 15:02 DAlperin

@lildude this is my first PR to linguist so let me know if I have made any glaring errors :)

DAlperin avatar Feb 11 '22 15:02 DAlperin

Looks good from a quick squiz, however usage is still really low for both extensions, even when considering our temporarily relaxed criteria. Labelling as such.

lildude avatar Feb 11 '22 16:02 lildude

Fixed the capitalization. As for the usage, fair. It just hit 1.0 yesterday which is how I came across it. Hopefully it will pick up soon.

DAlperin avatar Feb 11 '22 16:02 DAlperin