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 list of currently supported languages #659

Merged
merged 1 commit into from
Dec 18, 2018
Merged

Add list of currently supported languages #659

merged 1 commit into from
Dec 18, 2018

Conversation

kuba--
Copy link
Contributor

@kuba-- kuba-- commented Dec 12, 2018

Signed-off-by: kuba-- [email protected]

Closes #658

@kuba-- kuba-- requested a review from ajnavarro December 12, 2018 23:36
@kuba-- kuba-- added the documentation This is a documentation issue or pull request label Dec 12, 2018
@ajnavarro
Copy link
Contributor

Shall we wait til bblfsh/documentation#191 is resolved? and just point to that documentation?

@kuba--
Copy link
Contributor Author

kuba-- commented Dec 13, 2018

@ajnavarro @smola - how about now (I've updated a list, grouped by type and added links to enry and linguist)

@kuba-- kuba-- requested a review from smola December 13, 2018 12:51
@smola
Copy link
Contributor

smola commented Dec 17, 2018

This needs clafication. LANGUAGE function supports everything supported by src-d/enry. UAST supports whatever bblfsh has drivers for (if they are installed).

@kuba--
Copy link
Contributor Author

kuba-- commented Dec 17, 2018

@smola PTAL

- Textile
- reStructuredText

More details about aliases, groups, extensions, etc. you can find in [enry's repo](https://github.com/src-d/enry/blob/master/data/alias.go),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe all these explanations should come first, and the language list last? Because since it's very long, people may miss the distinction between enry-supported and bblfsh-supported.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'll leave just links and remove the list of all languages - WDYT?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, updated and rebased

@ajnavarro ajnavarro merged commit 2dfe135 into src-d:master Dec 18, 2018
@kuba-- kuba-- deleted the doc-658/langs branch December 18, 2018 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a documentation issue or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants