Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

fix: wrong interpolation (#1042) | chore: add debug traces for the webhook payload (#1043) | chore: add debug info for the payload (#1044) | fix: typo in parameters (#1045) | fix: incorrect variable name (#1046) | fix: do not use GT_REPO va #1057

Merged
merged 6 commits into from
Apr 20, 2021
Merged

fix: wrong interpolation (#1042) | chore: add debug traces for the webhook payload (#1043) | chore: add debug info for the payload (#1044) | fix: typo in parameters (#1045) | fix: incorrect variable name (#1046) | fix: do not use GT_REPO va #1057

merged 6 commits into from
Apr 20, 2021

Commits on Apr 20, 2021

  1. fix: wrong interpolation (elastic#1042)

    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    74cb112 View commit details
    Browse the repository at this point in the history

  2. chore: add debug traces for the webhook payload (elastic#1043)

    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    7cea2c9 View commit details
    Browse the repository at this point in the history

  3. chore: add debug info for the payload (elastic#1044) 

    * fix: do not use variable names as variable prefix

* chore: print payload

* fix: perform another API call to fetch PR data

We need a second API request, as the issue_comment API does not retrieve
data about the pull request. See
https://docs.github.com/en/developers/webhooks-and-events/webhook-events-and-payloads#issue_comment
    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    064ab48 View commit details
    Browse the repository at this point in the history

  4. fix: typo in parameters (elastic#1045)

    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    bb951b4 View commit details
    Browse the repository at this point in the history

  5. fix: incorrect variable name (elastic#1046)

    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    320071d View commit details
    Browse the repository at this point in the history

  6. fix: do not use GT_REPO variable, use fixed repo name instead (elasti… 

    …c#1049)

* fix: do not use GT_REPO variable, use fixed repo name

Because the repo is always Kibana, let's use it

* chore: remove unused echo
    @mdelapenya
    mdelapenya committed Apr 20, 2021
    Configuration menu
    Copy the full SHA
    c410690 View commit details
    Browse the repository at this point in the history