thollander/actions-comment-pull-request

GitHub action to comment pull request

actions-comment-pull-request's Issues

Abstain from renaming branches

First: Thank you for all the work you put into this.

One more thing: Please abstain from renaming branches in the future. The github syntax forced us to name a branch when using your github action. So we hardwired "master", and now we have to go through 30 projects to rename it to "main". Actually I would suggest to just stick with conventions or at least not change directions.

incompatible with lockfileversion2

npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but package-lock.json was generated for lockfileVersion@2. I'll try to do my best with it!

it works but takes about 10sec to comment whereas alternatives like mshick/add-pr-comment take less than a second.

Thollander's (11sec taken)	Mshic's (0sec taken)

Github actions Verified creator

Because this actions does not have a verified creator batch, github action workflow throws an error and won't proceed.
"thollander/actions-comment-pull-request@8c77f42 are not allowed to be used"

Could you please submit for publisher verification - https://docs.github.com/en/apps/github-marketplace/github-marketplace-overview/applying-for-publisher-verification-for-your-organization

Wrong Release Name

Hi I think your release has the wrong title.
https://github.com/thollander/actions-comment-pull-request/releases/tag/v2.5.0

Title says v2.4.3 but tag is v2.5.0 I think this might confuse people and think 2.4.3 is actually the latest tag?

How do you comment only once when opening PR?

Otherwise with this it comments on EVERY push to a branch assoc with an open PR =/

Parameter token or opts.auth is required

I am encountering an error message that says "Parameter token or opts.auth is required."

Run thollander/actions-comment-pull-request@v2
  with:
    filePath: /tmp/test.md
    comment_tag: nrt_file
    reactions: eyes, rocket
    mode: recreate
    create_if_not_exists: true
  env:
    CALLBACK_ENABLE: true
Error: Parameter token or opts.auth is required

Split PR comment if it is too long

Version:

thollander/actions-comment-pull-request@v2

Issue:

Run thollander/actions-comment-pull-request@v2
Error: An error occurred trying to start process '/home/runner/runners/2.310.2/externals/node16/bin/node' with working directory '/home/runner/work/somerepo/somerepo'. Argument list too long

The comment I am trying to add to PR is terraform plan, which size is bigger than can be added.

Possible solution

Ideally, PR comment might be divided into several parts and added one by one in PR

What's the license of the project?

Hello, I'd like to use the action provided by your project but I'd first like to confirm the license.
Can you please clarify by creating a LICENSE file containing the license terms? A MIT license would be great for me.

Upload images

Hey,

great repo and action. I was wondering how hard it would be to add the option to also upload images as a comment into the PR conversation.

Proper versionning

Context

Currently, versioning is managed thanks to pushing tags with the version number and updating the main branch.
It involves that if someone wants to have always the latest version updated (without breaking change), it's not possible.

Target

Manage a major tag that will help users to be updated everytime (eg v1 pointing to latest version v1.0.1 for instance).
And always update this tag when pushing a non breaking update.

Docs : https://docs.github.com/en/actions/creating-actions/about-actions#using-tags-for-release-management

Can I use this action to comment on PRs coming from a fork?

Hi,

Thanks for this action!

I have been able to use the action to comment on PRs coming from a fork (thanks for documenting the pull_request_target event).

I my comment I would need to use the fork repository, not the target.

Do you know what I could use to get the equivalent for ${{ github.repository }} on the fork? After I switched to pull_request_target, that variable became the target repository...

https://github.com/mwouts/jupytext/blob/main/.github/workflows/step_comment-pr.yml#L26

Thanks !

Question/Request: Is it possible to delete a comment without creating a new one?

First of all thanks for creating this GitHub Action! 🎉

Context

I've been working on making our pipeline check clearer in pull request. One of the ways of doing this is to comment some of the failing reasons.

Question/Request

I would like to remove an earlier created comment by this GitHub Action when for example the errors are fixed and the pipeline succeeds. Preferable in a way where I can run this action and have it succeed even if no comment was posted before. I assume it would be based on the comment_tag.

What do you think?

Cannot utilize environment variable. Can you provide an example?

When filePath is absolute GITHUB_WORKSPACE shouldn't be joined

When passing '/tmp/foobar.txt' as filePath (filePath: /tmp/foobar.txt) I got the error
Error: ENOENT: no such file or directory, open '/home/runner/work/user/project/tmp/foobar.txt'

actions-comment-pull-request/src/main.ts

Line 37 in d117ef6

_filePath = path.join(GITHUB_WORKSPACE, filePath);

Is it possible to attach a file to the comment?

Allow to silently exit on no PR error

Hey,

Currently, when the action is run on non-PR event the condition below is triggered which exits the action but also throws an error, which is really non-blocking (when used with continue-on-error). In my case, that message confuses users seeing the logs.

actions-comment-pull-request/src/main.ts

Line 37 in 8c77f42

core.setFailed('No issue/pull request in input neither in current context.');

I could of course add an if: github.ref == 'refs/heads/main' && github.event_name != 'pull_request' on this action step, but then the name of main branch is configurable in GitHub, so this is not really a bulletproof solution.

Would be good if we could decide (via input), whether the message is shown or not.

I would be glad to propose a PR for this change if this seems like a good idea.

Deletion should delete new comment, not the "updated" one

The deletion does not work if an update happened (of a comment created an another workflow).

Steps:

1. Workflow A creates comment
2. Workflow A ends
3. Workflow B starts (later)
4. Workflow B creates new comment (instead of updating existing one)
5. Workflow B post Action reports successful deletion. Comment from Workflow A is remived. Comment from Workflow B is still there.

Non-MWE:

JabRef/jabref#10057

State after post action of workflow B

Fork repo workaround

Hi, I implemented a workaround for the forked repos. I did it using bash / curl / jq, so it went much faster than a full blown action implementation. I think you might be interested in either adapting my approach for your action, or possibly create a new one? See https://github.com/nyurik/auto_pr_comments_from_forks

Custom Pr_number after merge to master not working

Hello!

I have got simple workflow:

on:
  workflow_dispatch:
    inputs:
      action:
        required: true
        default: "deploy"
        type: choice
        options:
        - test1
        - test

  push:
    branches:
      - master

      - name: Comment to PR with action link 
        uses: thollander/actions-comment-pull-request@v2
        with:
          message: |
             Deploy Begins! Actions URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
          pr_number: ${{ github.event.pull_request.number }} 
          GITHUB_TOKEN: ${{ secrets.COMMENT_TOKEN }}

After merge to master, I need to post commit on PR, which was merged to master. In action log I see, that it's now pr_number env:

Run thollander/actions-comment-pull-request@v2
  with:
    message: Deploy Begins! Actions URL: https://github.com/*]
  
    GITHUB_TOKEN: ***
    mode: upsert
    create_if_not_exists: true
Error: No issue/pull request in input neither in current context.

Is any ideas, why pr_number did not set?

Thank you!

Error when the PR is sent from a fork

It doesnt comment on the Pull Request when the PR is made from a fork, it only works when a PR is made from a branch within the repository.

On pull_request_target information

Hi! I only want to explain my problem and the solution, to see if it can be documented in the project and help to others.

I was stuck with the error Resource not accessible by integration and adding the write permission as explained in the readme of your project didn't work. Looking at the GitHub documentation, when the PR comes from a fork, it will have only read permission despite the permissions given in the action for the pull_request event.

The solution was to use the pull_request_target event. With this event, I can give write permissions without problem. The difference is that it will execute the action from the "target" branch and not from the origin PR. But this is usually good for the major part of projects.

This is my code change, if you want to look at it: betaflight/betaflight#12851

Maybe this information can be added to the readme to help others like me.

Regards and thanks for your project! It's very useful.

How do you add new line in comment?

I'm not able to add a new line in a comment, except by adding the tag <br>. It doesn't take \n or \r.

Comment content from a file

I'd like to be able to populate comment body with a content of a file.

Example use case:

• publish test coverage results

Available workarounds:

• use some env variable as a proxy.

Issue interpolating multiline outputs into the comment

Hi!

I'm currently trying to add a comment to a PR with the results of an end-to-end snapshot test. The results are generated in a prior step and I interpolate the step's output into the message field of this action, but am struggling to get the comment to keep its multilined formatting.

Here is a minimal reproduction of my workflow:

name: snapshot test
on: pull_request
jobs:
  api-regression-test:
    runs-on: ubuntu-latest
    steps:
      # [...steps to generate json snapshots and install the jd diffing utility]
      - name: Diff snapshot against canonical
        id: diff
        run: |
          # I've verified in CI that this output is being saved with multiple lines
          # I have also tried the >> $GITHUB_ENV method in referenced the github docs, with the same results
          SNAPSHOT_DIFF=$(jd current.json canonical.json)
          SNAPSHOT_DIFF="${SNAPSHOT_DIFF//'%'/'%25'}"
          SNAPSHOT_DIFF="${SNAPSHOT_DIFF//$'\n'/'%0A'}"
          SNAPSHOT_DIFF="${SNAPSHOT_DIFF//$'\r'/'%0D'}"
          echo "snapshot-diff=$SNAPSHOT_DIFF" >> $GITHUB_OUTPUT
      - name: Comment diff onto PR
        uses: thollander/actions-comment-pull-request@v2
        with:
          message: |
            ```diff
            [...some metadata about diff]
            ${{ steps.diff.outputs.snapshot-diff }}
            ```

And here is the output that I get:

[...some metadata about diff]
@ ["loc1","loc2",loc3]%0A- {"key1":{"key2": "val1"}}%0A @ ["loc1","loc2",loc3]%0A- {"key1":{"key2": "val1"}}

Converting \n characters to '%0A', as I'm doing in the diff step, is required to get outputs to not collapse onto one line. It also seems like @actions/core does this under the hood, as well.

But, the PR comment is outputting %0A where a newline should be. I'm not sure how to go about deserialising this output in the PR comment action in a way that allows for proper formatting. Is this something that there is an established pattern for?

Possibility to comment with a clickable images / badges with a link

Hello, really like the action, but it looks like it is not possible to comment with a clickable image / badge smth. like

[![Contribute](https://img.shields.io/badge/Eclipse_Che-Hosted%20by%20Red%20Hat-525C86?logo=eclipse-che&labelColor=FDB940)](https://workspaces.openshift.com/)

in the comment, it should resolve to the following clickable badge

Currently, it is not parsed correctly by the action. Any pointers. Thanks!

Node.js 16 actions are deprecated

Basically a warning given by GitHub action execution.

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: thollander/actions-comment-pull-request@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

[FEATURE] Add ability to delete and re-create current comment

Hello,
it will be good if possible to add the behaviour when the comment can be not just replaced, but removed and created again.

Sometimes with long commit history in feature branch it is not obvious is comment actual if one is somewhere on the top.
If we have ability to "move" comment to the last commit which "triggered" comment update it probably can be a little bit more obvious.

Any feedback is appreciated.

filePath content as code

when using filePath I would like the rendered file to be shown as code e.g. a yaml file shown as yaml in a code block

Using branches-ignore with comment-pull-request runs workflow when branch is set to be ignored

Hi all,
Currently I'm trying to comment on pull request to a master branch and using the "branches-ignore". I'm uncertain if this is github not running my workflow correct or my config. Can you steer me in the right direction?

• If I open a hotfix/update branch and pull request to master, the workflow runs and places the comment in the PR when it's set to be ignored and no comment should be inserted.

This is the workflow-

name: 'Comment on PR'
on:
pull_request:
branches-ignore:
- 'release/**'
- 'hotfix/**'

jobs:
example_comment_pr:
runs-on: ubuntu-latest
name: comment PR if needed
steps:
- name: Checkout
uses: actions/checkout@v3
- name: Comment PR
uses: thollander/actions-comment-pull-request@v1
with:
message: 'Thank you for the master PR. We only accept hotfix PRs for master branch. Please switch base branch to dev to get reviews'
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

• the open test PR is here jeffvg/mui5-react17#82

thank you

Failing to add html file as comment on PR from forked repository

I have a template repository and I'd like to add a prettier check before submitted PRs are accepted. Not only that, I'd like prettier to run and, case failed, to generate a diff output of which files should be changed for the PR to be accepted.

I was able to create such a workflow, where prettier runs and when failed an html diff is created. I was trying to include this html file as a comment on the PR with your action, but I couldn't. I saw in the docs that you suggest using pull_request_target, but in GitHub docs they have a warning about using this and pwn PRs.

Any idea of how to work it out?

How to link to a file created in previous step of a github action?

I am running pytest like this in one of the steps:
pytest --html=test_rounding_results/report.html -s --capture=tee-sys test_rounding.py
So this test is triggered if there is a pull request. It stores the report.html in the test_rounding_results folder. However this is just part of the workspace runner (and is later uploaded as an artifact).
Is it possible to link to this file somehow from the pull request comment created by the github-bot?
Alternatively, is it possible for the action to "commit" the file and then link to it?

[v2.3.1] Resource not accessible by integration

First of all, thank you very much for developing such a useful and convenient Actions.

I was using v2.3.0 and it worked very well and helped me simplify some of my work. But just now, @dependabot opened a new pull request yusancky/AllUp-Satwiki#9 in my repository, which wants to upgrade this Actions to v2.3.1. However, the pull request did not pass the test and no comment was given - it failed in The comment gives an error.

Bump thollander/actions-comment-pull-request from 2.3.0 to 2.3.1 #43

Error: Resource not accessible by integration

Here's part of the workflow file. This workflow runs on Ubuntu-latest.

      - name: Preview on Pull Request
        if: ${{ github.event_name == 'pull_request' }}
        uses: thollander/[email protected]
        with:
          message: |
            # 预览
            ```go
            ${{ steps.fetch_make.outputs.ALLUP_CONTENT }}
            ```
            > **Note**
            > 上方的 wikitext 代码块因展示高亮便利而使用 Go 语言高亮器，敬请理解。
          reactions: eyes
          comment_tag: actions-wikitext-preview

Changing icon and commenter name?

Is there any way to change the commenter name, icon and/or URL that name and icon link to?

Formatting of message

It would be great if you could apply some formatting to the comment such a code block, especially when it comes from a file

On `delete` mode new comment always will be created

Hello,
i have the situation when i change mode dynamically based on the result of another step, but on delete mode extra comment will be temporarily added all the time due to current logic.

I use this Action to add comment on PR in case if TFLinter finds some concerns based on exit code.
When TFLinter finds concern, the Action adds comment as it runs based on

      - name: 'Set comment PR mode'
        run: |
          if [ "${{ steps.tflint.outputs.exitcode }}" != "0" ]; then
            echo "pr_comment_mode=recreate" >> $GITHUB_ENV 
          else
            echo "pr_comment_mode=delete" >>  $GITHUB_ENV
          fi

      - name: 'Comment PR'
        if: ${{ steps.tflint.outputs.exitcode != '0' || env.pr_comment_mode == 'delete'}}
        uses: thollander/actions-comment-pull-request@v2
        with:
          message: |
            ## :microscope: TFLint validation
            ```shell
            ${{ steps.tflint.outputs.stdout || 'TFLint is running...' }}
            ```
          pr_number: ${{ github.event.pull_request.number }}
          mode: ${{ env.pr_comment_mode }}
          comment_tag: "tflint_comment"
          GITHUB_TOKEN: ${{ secrets.SOME_TOKEN }}

so, if there is any concerns it will be pr_comment_mode=recreate means comment will be added. Everything as expected.
When there is a new commit with changes, but still some concerns from TFLinter, comment will be recreated and Action works as expected.
But if all the TFLint concerns were fixed, then mentioned step sets pr_comment_mode=delete .
In this situation there is a small side effect as delete mode always creates comment. The thing is, there is a situation when it will be two PR comments from this Action, one created when it was TFLint concerns and the new extra short living comment as createComment function will be executed before post step which will remove both comments later.

It will be great if it can be avoided the creation of new intermediate PR comment when mode: delete in the situation I've described.

Maybe, it is kind of "new" mode (cleanup ?) when delete works just as delete, without creation of any comment before the deletion in the end during post step.

Deleting the created comment?

I want to delete the comment as the last step on a workflow.

It doesn't look like that functionality exists, is there another way I could do it?

Delete a Comment Without Creating a New One

Desired functionality:

1. Workflow runs, creating a comment with tag foo
2. Workflow runs again, deletes the comment with tag foo

Current functionality

1. Workflow runs, creating a comment with tag foo
2. Workflow runs again, deletes the comment with tag foo
3. New comment is created in deletion step
   a. Ideally, you would not have to create a new comment upon deletion of another comment

Here's an example workflow snippet

      - name: Post Comment
        uses: thollander/actions-comment-pull-request@v2
        with:
          message: |
            #### hello world
          comment_tag: foo
          mode: recreate

      - name: Delete Comment
        uses: thollander/actions-comment-pull-request@v2
        with:
          message: |
            #### Why do I need this to delete?
          comment_tag: foo
          mode: delete
          create_if_not_exists: false # thought this may fix it (it does not :/)

At the end of this, the Post Comment comment is deleted, but seemingly always replaced with a new comment that has the contents of the message in the Delete Comment step.

Ideally, there would be a way to delete an existing comment without creating a new one.

If there is a way to do this and I'm missing something here, please let me know

Thanks!

Update: seems this functionality may not be supported by GitHub: https://github.com/orgs/community/discussions/26256

Update a specific comment that was not created by this action?

Hello,

i'm interested in using this action, and i would like to use it to update a comment that was already created.

My use case is as follows:

• user comment on a PR with a keyword
• I trigger a workflow with issue_comment
• I use https://github.com/Khan/pull-request-comment-trigger to verify the keyword, if the keyword matches, the workflow goes on
• I would like to update the comment that triggered issue_comment by appending text to it

Is that something that would fall under the scope of this project, and you would be willing to add as a new feature?

Thanks

HttpError: Resource not accessible by integration

Config: https://github.com/DIYgod/RSSHub/blob/master/.github/workflows/dependabot-fork.yml
Log: https://github.com/liyishuai/RSSHub/runs/5237459528?check_suite_focus=true#step:4:8

(node:1) UnhandledPromiseRejectionWarning: HttpError: Resource not accessible by integration
at /node_modules/@octokit/request/dist-node/index.js:66:23
at processTicksAndRejections (internal/process/task_queues.js:95:5)

Question: how to use a file content as comment messages on a PR?

Hi, I'm trying to auto generate comments for new PRs in my repository.

I found this action, it looks like the filePath option can help me do these things:

• read the content from a file, for example: pr-template.md file (it has a markdown content, and it is complicated and cannot be directly put into this action as a message).
• use the content from the file as a message.

Here's my pr.yml actions:

name: PR
on:
  pull_request: ~
  push:
    branches:
      - main
jobs:
  pr_comment:
    name: PR comment
    runs-on: ubuntu-latest
    steps:
    - name: Checkout
      uses: actions/checkout@v4
    - name: ls -lha
      shell: bash
      run: |
          echo "ls -lha ."
          ls -lha .
    - name: PR Comment
      uses: thollander/actions-comment-pull-request@v2
      with:
        filePath: pr-template.md

But there's an error in my workflow:

https://github.com/linrongbin16/fzfx.nvim/actions/runs/6439036401/job/17486263012?pr=244

The error is:

Error: Resource not accessible by integration

Would you please help me?

Populate an output variable with the input message

I want to use the message from the PR to populate the Github Job Summary as well. Ideally, the action should generate an output variable that I can use in later steps. It would be even better if this could include the URL of the comment as another variable.

Thanks for the library; it's really useful.

[Feature] Ability to specify a comment should NOT be created if its tag does not yet exist

I have a workflow that checks the codebase for the existence of some token, and then checks a file within the codebase for a baseline of that token. In some cases, the token is removed but the baseline remains, and we want to notify PR authors that they can remove the baseline. We do this with a comment:

      - name: Comment on PR
        uses: thollander/actions-comment-pull-request@c22fb302208b7b170d252a61a505d2ea27245eff #v2.0.0
        if: ${{ steps.check_for_useless_baselines.outputs.useless_baselines == 'true' }}
        with:
          message: Violations exist <tag>
          comment_tag: <tag>

I would love to be able to tell the author that they have resolved all the issues, which would be based on the following conditional in a separate step that looks much like the one above:

if: ${{ steps.check_for_useless_baselines.outputs.useless_baselines == 'false' }}

However, I don't want to post this comment unless the first one has been posted (i.e unless the author had previous violations in a past commit), and since my workflow can't know the state of previous runs, I feel it would need to be something like:

      - name: Comment on PR
        uses: thollander/actions-comment-pull-request@c22fb302208b7b170d252a61a505d2ea27245eff #v2.0.0
        if: ${{ steps.check_for_useless_baselines.outputs.useless_baselines == 'false' }}
        with:
          message: Violations fixed <tag>
          comment_tag: <tag>
          create_if_not_exists: false # defaults to true

The end result, when violations existed, would be:

1. Comment indicating violations exist
2. Comment indicating violations were resolved

The end result, when no violations existed, would be that no comments are posted.

How I can get a output of a file commented ?

I would like to put in a comment the content of a file

Resource not accessible by integration

Not an expert on github actions, something obvious i'm missing?

In my github actions log:

Run thollander/[email protected]
  with:
    GITHUB_TOKEN: ***
    pr_number: 1
    filePath: /home/runner/work/llvm-project/llvm-project/clang-format.patch
    mode: upsert
    create_if_not_exists: true
Error: Resource not accessible by integration

the code:

    - name: Comment Pull Request
      uses: thollander/[email protected]
      with:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        pr_number: ${{github.event.number}}
        filePath: ${{ github.workspace }}/clang-format.patch

i seem to recall that the default github token has access to everything, is that not the case?

README update

It is great to see that you've moved to the main branch terminology!

Can you please update your README to reflect that? Thank you so much!

- name: Comment PR
  uses: thollander/actions-comment-pull-request@main

Change `comment_includes` to be automatic

Improvement idea: switch from comment_includes to comment_tag

If message body had something like this appended <!--- thollander/actions-comment-pull-request "comment_tag" --->
it would be easy to precisely select comment for editing and wouldn't be visible to user.

Benefits:

• independent of message body
• no false-positive matches
• not displayed to user

Unable to resolve `master` version

Is the master now main?
I get this error when using the action.
Error: Unable to resolve action thollander/actions-comment-pull-request@master, unable to find version master
If so, then please update the readme in the Usage section.

Failure with NPM and stack overflow while building this action image

Today we started getting the following error during our workflows, not sure if this is an npm issue or the package-lock.json in this repo:

Build container for action use: '/home/runner/work/_actions/thollander/actions-comment-pull-request/master/Dockerfile'.
  /usr/bin/docker build -t 1e5c35:ad639c88d707402b8e5ba2e8e25386b6 -f "/home/runner/work/_actions/thollander/actions-comment-pull-request/master/Dockerfile" "/home/runner/work/_actions/thollander/actions-comment-pull-request/master"
  Sending build context to Docker daemon  36.35kB
  
  Step 1/4 : FROM node:slim
  slim: Pulling from library/node
  babf97a3f00a: Pulling fs layer
  d81eb1809d95: Pulling fs layer
  54b0718e1fd5: Pulling fs layer
  9f80bf52a6c0: Pulling fs layer
  28724063fd29: Pulling fs layer
  9f80bf52a6c0: Waiting
  28724063fd29: Waiting
  d81eb1809d95: Verifying Checksum
  d81eb1809d95: Download complete
  9f80bf52a6c0: Verifying Checksum
  9f80bf52a6c0: Download complete
  babf97a3f00a: Verifying Checksum
  babf97a3f00a: Download complete
  28724063fd29: Verifying Checksum
  28724063fd29: Download complete
  54b0718e1fd5: Verifying Checksum
  54b0718e1fd5: Download complete
  babf97a3f00a: Pull complete
  d81eb1809d95: Pull complete
  54b0718e1fd5: Pull complete
  9f80bf52a6c0: Pull complete
  28724063fd29: Pull complete
  Digest: sha256:a39f2854f7fc02c8636c11227ca2d6680bd46a2173168650928b69cc03dffb1c
  Status: Downloaded newer image for node:slim
   ---> 2870b8b46426
  Step 2/4 : COPY . .
   ---> e1de5efdbd73
  Step 3/4 : RUN npm install --production
   ---> Running in 5263be933aac
  npm notice 
  npm notice New patch version of npm available! 7.0.2 -> 7.0.3
  npm notice Changelog: <https://github.com/npm/cli/releases/tag/v7.0.3>
  npm notice Run `npm install -g [email protected]` to update!
  npm notice 
  npm ERR! Maximum call stack size exceeded
  
  npm ERR! A complete log of this run can be found in:
  npm ERR!     /root/.npm/_logs/2020-10-22T19_01_38_190Z-debug.log
  The command '/bin/sh -c npm install --production' returned a non-zero code: 1
  
  Warning: Docker build failed with exit code 1, back off 2.14 seconds before retry.
  /usr/bin/docker build -t 1e5c35:ad639c88d707402b8e5ba2e8e25386b6 -f "/home/runner/work/_actions/thollander/actions-comment-pull-request/master/Dockerfile" "/home/runner/work/_actions/thollander/actions-comment-pull-request/master"
  Sending build context to Docker daemon  36.35kB
  
  Step 1/4 : FROM node:slim
   ---> 2870b8b46426
  Step 2/4 : COPY . .
   ---> Using cache
   ---> e1de5efdbd73
  Step 3/4 : RUN npm install --production
   ---> Running in 8e917ec8efea
  npm notice 
  npm notice New patch version of npm available! 7.0.2 -> 7.0.3
  npm notice Changelog: <https://github.com/npm/cli/releases/tag/v7.0.3>
  npm notice Run `npm install -g [email protected]` to update!
  npm notice 
  npm ERR! Maximum call stack size exceeded
  
  npm ERR! A complete log of this run can be found in:
  npm ERR!     /root/.npm/_logs/2020-10-22T19_01_41_950Z-debug.log
  The command '/bin/sh -c npm install --production' returned a non-zero code: 1
  
  Warning: Docker build failed with exit code 1, back off 7.085 seconds before retry.
  /usr/bin/docker build -t 1e5c35:ad639c88d707402b8e5ba2e8e25386b6 -f "/home/runner/work/_actions/thollander/actions-comment-pull-request/master/Dockerfile" "/home/runner/work/_actions/thollander/actions-comment-pull-request/master"
  Sending build context to Docker daemon  36.35kB
  
  Step 1/4 : FROM node:slim
   ---> 2870b8b46426
  Step 2/4 : COPY . .
   ---> Using cache
   ---> e1de5efdbd73
  Step 3/4 : RUN npm install --production
   ---> Running in 9362d4ee85ea
  npm notice 
  npm notice New patch version of npm available! 7.0.2 -> 7.0.3
  npm notice Changelog: <https://github.com/npm/cli/releases/tag/v7.0.3>
  npm notice Run `npm install -g [email protected]` to update!
  npm notice 
  npm ERR! Maximum call stack size exceeded
  
  npm ERR! A complete log of this run can be found in:
  npm ERR!     /root/.npm/_logs/2020-10-22T19_01_50_702Z-debug.log
  The command '/bin/sh -c npm install --production' returned a non-zero code: 1
  
Error: Docker build failed with exit code 1

Get `pr_number` when triggered by `issue_comment`

I am using https://github.com/Khan/pull-request-comment-trigger with just on: issue_comment: types: [created] to trigger the Action when I write my command on the PR as a comment. I don't use pull_request: types: [opened] as I don't want it to mess up every PR with those checks.

However, I had this error:

I fixed it by adding this, as this guy said: actions/checkout#331 (comment)
pr_number: ${{ github.event.issue.number }}

So, I think this action should try to get this var if the current one isn't available to handle the issue_comment event.