X Tutup
The Wayback Machine - https://web.archive.org/web/20210725052523/https://github.com/github/linguist/issues/5013
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 Cuelang #5013

Closed
4 tasks done
leoluk opened this issue Sep 19, 2020 · 5 comments
Closed
4 tasks done

Add support for Cuelang #5013

leoluk opened this issue Sep 19, 2020 · 5 comments

Comments

@leoluk
Copy link
Task lists! Give feedback

@leoluk leoluk commented Sep 19, 2020

Preliminary Steps

Please confirm you have...

Problem Description

Linguist should support for https://cuelang.org/docs/about, an upcoming data validation/configuration language that is already seeing widespread use.

Cue files are currently hard to read on GitHub: https://github.com/leoluk/NetMeta/blob/master/deploy/single-node/traefik/deployment.cue

There's a WIP pull request by @pchaigno which seems blocked by an unfortunate file extension collision that'll require heuristics: #4941

@myitcv
Copy link

@myitcv myitcv commented Mar 22, 2021

@lildude is there anything we can do to help move this one along?

@lildude
Copy link
Member

@lildude lildude commented Mar 22, 2021

@myitcv As #4941 appears to have been abandoned, the best thing to do is open a new PR, but this will need to include support for Cuelang and the other language identified.

From a quick look, it looks like all the other .cue files may be cue sheet which is quite different so adding both in the same PR shouldn't be too challenging. A grammar isn't needed for inclusion.

@myitcv
Copy link

@myitcv myitcv commented Mar 22, 2021

Thanks @lildude.

@zchee - you mentioned offline you'd be keen to work on this - did you make any progress? I'm happy to help out.

@myitcv myitcv mentioned this issue Mar 22, 2021
5 tasks
@zchee
Copy link
Contributor

@zchee zchee commented Mar 22, 2021

@myitcv ah, I forgot it. will send PR.

@lildude
Copy link
Member

@lildude lildude commented Apr 16, 2021

#5312 has been merged.

It just missed the last release so GitHub.com won't gain support until the next release is made which will be in about a month's time. As Linguist now has support, I'm going to close this issue. Thanks to everyone involved 🙇.

@lildude lildude closed this Apr 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
X Tutup