Skip to content
Snippets Groups Projects

GQL code owners Validation for every file named CODEOWNERS

What does this MR do and why?

GQL code owners Validation for every file named CODEOWNERS

Changelog: added EE: true

See #216066 (comment 1468152154)

Screenshots or screen recordings

    query {
      project(fullPath: PROJECT_FULL_PATH) {
        repository {
          validateCodeownerFile(ref: REF, path: BLOB_PATH) { # nil if the BLOB_PATH file name is not 'CODEOWNERS' or does not exists
            total
            validationErrors {
              code
              lines
            }
          }
        }        
      }
    }

Related to #415352 (closed)

Edited by Patrick Cyiza

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Patrick Cyiza requested review from @imand3r

    requested review from @imand3r

  • Allure report

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :exclamation: test report for dada2a35

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 47     | 0      | 0       | 40    | 47    | ❗     |
    | Create      | 27     | 0      | 1       | 20    | 28    | ❗     |
    | Govern      | 19     | 0      | 0       | 18    | 19    | ❗     |
    | Data Stores | 20     | 0      | 0       | 15    | 20    | ❗     |
    | Manage      | 12     | 0      | 1       | 12    | 13    | ❗     |
    | Verify      | 8      | 0      | 0       | 8     | 8     | ❗     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 133    | 0      | 2       | 113   | 135   | ❗     |
    +-------------+--------+--------+---------+-------+-------+--------+
    Edited by Ghost User
  • Ian Anderson approved this merge request

    approved this merge request

  • :wave: @imand3r, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Nataliia Radina requested review from @garyh

    requested review from @garyh

  • Patrick Cyiza added 1 commit

    added 1 commit

    • a94dcec9 - GQL code owners Validation for every file named CODEOWNERS

    Compare with previous version

  • Patrick Cyiza changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading