Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

[cosmetic] add the author in the slack message#999

Merged
mergify[bot] merged 2 commits intoelastic:masterfrom
v1v:feature/add-user-pr-when-slack-message
Feb 22, 2021
Merged

[cosmetic] add the author in the slack message#999
mergify[bot] merged 2 commits intoelastic:masterfrom
v1v:feature/add-user-pr-when-slack-message

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Feb 22, 2021

What does this PR do?

Add the author of a particular PR

Why is it important?

ChangeSet JSON schema changed and the slack template did not use the new format

Test

From the UTs, It created the below slack message

image

@botelastic botelastic Bot added the groovy label Feb 22, 2021
@v1v v1v self-assigned this Feb 22, 2021
@v1v v1v requested a review from a team February 22, 2021 12:21
@v1v v1v added the bug Something isn't working label Feb 22, 2021
@ghost
Copy link
Copy Markdown

ghost commented Feb 22, 2021

💚 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 #999 updated

  • Start Time: 2021-02-22T14:05:22.932+0000

  • Duration: 11 min 12 sec

  • Commit: ec694e6

Test stats 🧪

Test Results
Failed 0
Passed 1046
Skipped 2
Total 1048

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 1046
Skipped 2
Total 1048

@v1v v1v added backport-to-7.x Automatically backport PRs from master ready-to-merge labels Feb 22, 2021
@mergify mergify Bot merged commit 7b2253a into elastic:master Feb 22, 2021
mergify Bot added a commit that referenced this pull request Feb 22, 2021
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 7b2253a)
mergify Bot added a commit that referenced this pull request Feb 22, 2021
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 7b2253a)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Mar 18, 2021
* upstream/7.x:
  Fix the getCause and getCauses for the AbortException (#1041) (#1042)
  gsutil for windows with python (#1008) (#1009)
  Report only one DeleteDir step failure (#1006) (#1007)
  [cosmetic] Replace multiline chars in the GitHub PR comment (#998) (#1003)
  [cosmetic] add the author (#999) (#1002)
  chore: use opentelemetry release (#1000) (#1001)
  Avoid cache gsutils temporary location (#994) (#995)
  [JCasC] google cloud engine local setup (#979) (#992)
  Support curl in addition to wget for gsutil (#988) (#989)
  Support windows for gsutil (#985) (#987)
  Add googleStorageUploadExt step (#867) (#982)
  Add releaseNotification step to simplify agent release pipelines (#976) (#981)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-to-7.x Automatically backport PRs from master bug Something isn't working groovy ready-to-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants