-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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 Open Policy Agent as a new language #4371
Conversation
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.
Thanks for the neat pull request!
Unfortunately, as you noticed, usage is currently too low to meet our requirement of hundreds of repositories. We'll probably have to close this and reopen once adoption is high enough.
How old is OPA? Do you expect usage to grow in the near future?
Thanks for the quick reply.
Started in early 2016.
Yes, the end-user community has grown quite a bit over the last year and a half. There are a number of companies running OPA (in production) today. We're also seeing other open source projects embedding & building on top of OPA.
That's fair! We were hoping that since the growth is there and the extension was not taken, that an exception could be made 😅 I know lots of folks who have .rego files in private repos that would be happy to see this land :D |
Ok. I'll add it to #4219 when I get back to a decent workstation (probably next week); I'll flag as Pending popularity in the meantime. That way we can monitor progress on adoption and it shouldn't be long before we can support it in Linguist. |
Closing in favor of #4219. We'll monitor the popularity of the extension there. I'll try to check popularity again in ~6 months and I'll post here if usage grows enough to warrant support in Linguist. |
👋 @pchaigno could you re-run the popularity check? It's been a few months and we've seen growth (>200 repos now by my count). |
@pchaigno this PR is fine. The only thing I'm unsure of is the language ID in the PR. Does that need to be regenerated? |
No. If you used the script to generate the language ID the first time, there's no need to regenerate it. |
05233f0
to
a09c272
Compare
@pchaigno I've amended the commit to resolve the conflicts and fix the test failures. Let me know if anything else has to be done to land this. |
Could you link in the template to where exactly you took the sample files from? Otherwise, LGTM. |
Open Policy Agent (OPA) implements a domain-agnostic policy language for the cloud-native ecosystem. It is owned by the Cloud Native Computing Foundation. It embodies policy-as-code, which means organizations often store, review, test, and revise their policies on GitHub. https://www.openpolicyagent.org/ https://www.cncf.io/ https://github.com/search?q=extension%3Arego+NOT+nothack&type=Code Signed-off-by: Torin Sandall <[email protected]>
@pchaigno done! I updated the k8s one because that changed in the OPA docs. Should be ready to merge now. |
@pchaigno can this be merged now? |
👋 hello!
I'd like to add syntax support for the Open Policy Agent (OPA) project. OPA is hosted by the Cloud Native Compute Foundation (CNCF). OPA implements a domain-agnostic policy language. It embodies policy-as-code, which means organizations often store, review, test, and revise their policies on GitHub.
Checklist:
79252 public repositories (as of this PR).