grantbirki / comment

GitHub Action to create, update, or add a reaction to any issue or pull request

@GrantBirki

Hello! My name is Grant and I am a Senior Security Engineer at @GitHub.
Previously, I was a security engineer working on CDNs and deployment pipelines for @Nordstrom.

🔭 I'm currently working on: Automation, security, devops, several open source projects, public Actions workflows, and all things chatops
🌱 I'm currently learning: Rust, JS, and custom GitHub Actions
💎 I love Ruby and even maintain a few open source libraries (Gems)
🧑‍🔬 A core contributor for the-hideout and github/branch-deploy

I love to build, learn, support the open source community, and have fun along the way. ❤️:octocat:

Open source projects ⭐

🛠 Projects	⭐ Stars
branch-deploy
combine-prs
tarkov-api
tarkov-dev
stash
ghtrending
auditor-action
git-diff-action
json-yaml-validate
issue-template-parser
comment
Fastly Tempo
errbot
upgraded-waffle
cah-generator

Where to find me 📍

Stats 📊

comment's Issues

feature request: support addint reactions to PR reviews / PR review comments

Currently, there are various ways to leave comment-like messages on GitHub PRs:

• PR comments
• PR reviews
• PR review comments

For our particular use-case, we'd like to be able to add reactions to all 3 of these types, but it seems that this action doesn't currently support PR reviews / PR review comments.

The following minimized workflow attempts to add a reaction to a PR review comment:

on:
  pull_request_review_comment:
    types: [created, edited]

jobs:
  add_reaction:
    runs-on: ubuntu-latest
    steps:
      - name: Add reaction
        uses: GrantBirki/[email protected]
        with:
          comment-id: ${{ github.event.comment.id }}
          reactions: rocket

However, it fails with:

Adding reaction 'rocket' to comment id 'XXXX' failed with HttpError: Not Found.

Add example for passing a list

Hi 👋

I found out today how to pass a list of arguments via the variables. I struggled a bit and this solution could be beneficial for others, in case you want to add it to the readme/docs.

Here's the setup:

jobs:
  example:
    runs-on: ubuntu-latest
    steps:
      - id: run
        run:  echo "files=a,b,c" >> "$GITHUB_OUTPUT"

      - uses: GrantBirki/comment
        with:
          # other config
          vars: |
            files: ${{ steps.run.outputs.files }}
          body: |
            # Files
            {% for file in files.split(",") %}
            - `{{ file }}`
            {% else %}
            - No file
            {% endfor %}

What's important here is to serialize the files list when passing between steps. Passing a multiline-string breaks the vars value.

Thanks ✨