[PR #2649] fix: job's GetMatrixes cannot get the correct matrix #2535

New issue

Open

opened 2026-03-01 21:55:51 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 21:55:51 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/2649
Author: @silky1313
Created: 2/7/2025
Status: 🔄 Open

Base: master ← Head: dev

📝 Commits (6)

8779bfc fix: job's GetMatrixes cannot get the correct matrix
093142e Merge branch 'master' into dev
27a5c24 feat: Add a matrix include to cover the test case
5ae9ef7 fix: Clear an empty test case
bf0817d fix: rename commonCheckAllOriginKey to commonAllOriginKeyMatch
162671c fix: problems caused by empty matrix

📊 Changes

3 files changed (+137 additions, -55 deletions)

View changed files

➕ pkg/model/testdata/strategy/matrix/coverage_test.yml (+39 -0)
📝 pkg/model/workflow.go (+54 -51)
📝 pkg/model/workflow_test.go (+44 -4)

📄 Description

#2648
This Pull requests is mainly to solve this issue

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/nektos/act/pull/2649 **Author:** [@silky1313](https://github.com/silky1313) **Created:** 2/7/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `dev` --- ### 📝 Commits (6) - [`8779bfc`](https://github.com/nektos/act/commit/8779bfce2ee80dbd39a9e31a2a7ba8ea3a38fc54) fix: job's GetMatrixes cannot get the correct matrix - [`093142e`](https://github.com/nektos/act/commit/093142e5336e84a876ac5ef1d8b96b50344ea7d9) Merge branch 'master' into dev - [`27a5c24`](https://github.com/nektos/act/commit/27a5c248db402d911bebb0bc73fb34c3d73e0835) feat: Add a matrix include to cover the test case - [`5ae9ef7`](https://github.com/nektos/act/commit/5ae9ef7e0fcda5c69472b2c55df856934be40dec) fix: Clear an empty test case - [`bf0817d`](https://github.com/nektos/act/commit/bf0817d60ea1490b0d2ec596ab8dc119f8597617) fix: rename commonCheckAllOriginKey to commonAllOriginKeyMatch - [`162671c`](https://github.com/nektos/act/commit/162671cb083609e531939db62cf6b176d6d5a376) fix: problems caused by empty matrix ### 📊 Changes **3 files changed** (+137 additions, -55 deletions) <details> <summary>View changed files</summary> ➕ `pkg/model/testdata/strategy/matrix/coverage_test.yml` (+39 -0) 📝 `pkg/model/workflow.go` (+54 -51) 📝 `pkg/model/workflow_test.go` (+44 -4) </details> ### 📄 Description #2648 This Pull requests is mainly to solve this issue --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem added the

pull-request

label

2026-03-01 21:55:51 +03:00

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/act#2535

No description provided.

Rows
Columns