From ba6e5aaec58bdd1f762ab94b10066df99c1ad862 Mon Sep 17 00:00:00 2001 From: Thomas Widhalm Date: Tue, 16 Jan 2024 14:12:08 +0000 Subject: [PATCH] Always run required checks (#301) We have quite sophisticated rules when to run which Molecule checks. Unfortunately it's quite easy to have a PR or push where the full stack checks won't run. But since we require these checks, you can end up in a situation where you don't have check results but can't merge without those results. So I simplified the rules when to run these checks. fixes #300 --- .github/workflows/test_roles_pr.yml | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/.github/workflows/test_roles_pr.yml b/.github/workflows/test_roles_pr.yml index fc3dab70..026d9126 100644 --- a/.github/workflows/test_roles_pr.yml +++ b/.github/workflows/test_roles_pr.yml @@ -13,23 +13,7 @@ on: - warning - debug pull_request: - branches: - - 'feature/**' - - 'fix/**' - - '!doc/**' - paths: - - 'roles/**' - - '.github/workflows/test_roles_pr.yml' - - 'molecule/elasticstack_default/**' push: - branches: - - 'feature/**' - - 'fix/**' - - '!doc/**' - paths: - - 'roles/**' - - '.github/workflows/test_roles_pr.yml' - - 'molecule/elasticstack_default/**' merge_group: jobs: