[PATCH] D116762: workflows: Make issue-subscriber more robust for labels with special characters

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 6 23:09:17 PST 2022


tstellar added a comment.

In D116762#3226065 <https://reviews.llvm.org/D116762#3226065>, @kwk wrote:

> Since this looks like the beginning of something new and big I highly suggest to consider using GraphQL instead of doing the many REST calls that are needed. Especially when you drive stuff from a github action you have almost everything you need right at your finger tips. I mean all the global IDs for issues and alike. I'm working on something for another project and here's my example of how one could encapsulate the github functionality into classes with easy usage: https://gist.github.com/kwk/c89b6a3e5eb40487fed78f226e982fcc#file-graphql-py-L138 .
>
> Anyway this is just a thought.

It would be nice if the PyGitHub module had wrappers for this already.  What are the advantages of GraphQL vs the REST API?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116762/new/

https://reviews.llvm.org/D116762



More information about the llvm-commits mailing list