-
Notifications
You must be signed in to change notification settings - Fork 90
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 material #607
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.
Sounds very good, many thanks for writing this! Only a minor comment on the OPA logo
This looks great! Thanks @mosabua |
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.
This looks great! Once you can have a look at the comments I left, I think we'll be good to go
@mosabua did you notice the artifacts on the logo that @sbernauer mentioned? There's some noise on the png |
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.
LGTM - Thanks a lot @mosabua !
ba32174
to
6ebd642
Compare
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.
I feel like the blog is kind of underselling what you can do with this, why you'd care, etc. The story is fun to have but shouldn't be the bulk of the content IMO
12bb3f7
to
ecf0165
Compare
I added a whole benefits section .. you think this is good now or did I miss something else @colebow ? |
- Add as tool - Add blog post with Trino Summit recap and update - Fix Bloomberg links in summit recap post Co-authored-by: Pablo Arteaga <[email protected]>
In the tools section and with a detailed blog post.
Also fyi @soenkeliebau