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 #1058

Conversation

mdelapenya
Copy link
Contributor

Backports the following commits to 7.12.x:

* 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
…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 self-assigned this Apr 20, 2021
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #1058 opened

  • Start Time: 2021-04-20T09:59:28.711+0000

  • Duration: 74 min 13 sec

  • Commit: 5526d7a

Test stats 🧪

Test Results
Failed 0
Passed 140
Skipped 0
Total 140

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 140
Skipped 0
Total 140

@mdelapenya mdelapenya merged commit cf76cd6 into elastic:7.12.x Apr 20, 2021
@mdelapenya mdelapenya deleted the backport/7.12.x/pr-1042_pr-1043_pr-1044_pr-1045_pr-1046_pr-1049 branch June 2, 2021 05:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants