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

isMemberOf step to query GitHub user/teams #711

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

v1v
Copy link
Member

@v1v v1v commented Sep 4, 2020

What does this PR do?

Add step to search for GitHub memberships for a given user and team.

Why is it important?

Filter PRs for the APM-UI e2e if their owners are members of the given GH teams

How to use it

elastic/kibana#76764

@v1v v1v self-assigned this Sep 4, 2020
@v1v v1v added the automation label Sep 4, 2020
@apmmachine
Copy link
Contributor

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #711 opened]

  • Start Time: 2020-09-04T12:01:11.264+0000

  • Duration: 6 min 23 sec

Test stats 🧪

Test Results
Failed 0
Passed 757
Skipped 9
Total 766

Steps errors

Expand to view the steps failures

  • Name: Archive JUnit-formatted test results
    • Description: [2020-09-04T12:04:00.961Z] Recording test results
      [2020-09-04T12:04:01.219Z] Failed to read /var/li

    • Duration: 0 min 0 sec

    • Start Time: 2020-09-04T12:04:00.951+0000

    • log

@v1v v1v marked this pull request as ready for review September 4, 2020 12:31
@v1v v1v requested a review from a team September 4, 2020 12:31
@v1v v1v added the enhancement New feature or request label Sep 4, 2020
@botelastic botelastic bot added the groovy label Sep 4, 2020
@v1v v1v merged commit 361240c into elastic:master Sep 4, 2020
@v1v v1v deleted the feature/validate-user-member-of-github branch September 4, 2020 14:18
v1v referenced this pull request in v1v/apm-pipeline-library-backup Sep 4, 2020
* upstream/master:
  isMemberOf step to query GitHub user/teams (#711)
  [test-infra] run docker if docker-machine installed (#710)
  super-linter seems to be broken (#709)
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.

3 participants