kyma-project / docker-registry

Apache License 2.0
2 stars 7 forks source link

Mark tests required #75

Closed halamix2 closed 3 weeks ago

halamix2 commented 3 months ago

Description

Currently tide will merge PR even if tests have failed.

We need to change that by

AC

Execution

Attachments https://docs.github.com/en/enterprise-server@3.11/actions/using-workflows/required-workflows

pPrecel commented 3 months ago

All of our tests have the paths-ignore configuration (example here and here) that allows us to push changes to the documentation without running and waiting for tests:

on:
  pull_request:
    paths-ignore:
      - 'docs/**'
      - 'examples/**'

We need to discuss what we expect regarding this configuration. If we want to mark tests as required, we need to run tests on every change.