[PR #1428] [MERGED] fix: keep path to event json file in composite actions #1944

Closed
opened 2026-03-01 21:53:21 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/1428
Author: @KnisterPeter
Created: 11/7/2022
Status: Merged
Merged: 11/16/2022
Merged by: @mergify[bot]

Base: masterHead: keep-event-json-path


📝 Commits (4)

  • 2ee1678 fix: keep path to event json file in composite actions
  • 0d1b4b8 test: add test case
  • 5afd507 test: paste the test correctly and revert a line
  • 7f36231 Merge branch 'master' into keep-event-json-path

📊 Changes

2 files changed (+18 additions, -1 deletions)

View changed files

📝 pkg/runner/action_composite.go (+1 -0)
📝 pkg/runner/testdata/pull-request/main.yaml (+17 -1)

📄 Description

The event.json paths need to be copied over, since it the
GithubContext is recreated from the composite RC. And that
does read some value from the event file if available.

Fixes #1401


🔄 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/1428 **Author:** [@KnisterPeter](https://github.com/KnisterPeter) **Created:** 11/7/2022 **Status:** ✅ Merged **Merged:** 11/16/2022 **Merged by:** [@mergify[bot]](https://github.com/apps/mergify) **Base:** `master` ← **Head:** `keep-event-json-path` --- ### 📝 Commits (4) - [`2ee1678`](https://github.com/nektos/act/commit/2ee167854026f254b1ec8f493bae0ba5cc0fdaa6) fix: keep path to event json file in composite actions - [`0d1b4b8`](https://github.com/nektos/act/commit/0d1b4b83ccf5e054743bdf226a852a7775acb87b) test: add test case - [`5afd507`](https://github.com/nektos/act/commit/5afd50707a3aae8a17cbe60b57410f8802a09eff) test: paste the test correctly and revert a line - [`7f36231`](https://github.com/nektos/act/commit/7f3623184a13461cdc46cbc50ea1e34f2cc9772c) Merge branch 'master' into keep-event-json-path ### 📊 Changes **2 files changed** (+18 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `pkg/runner/action_composite.go` (+1 -0) 📝 `pkg/runner/testdata/pull-request/main.yaml` (+17 -1) </details> ### 📄 Description The event.json paths need to be copied over, since it the GithubContext is recreated from the composite RC. And that does read some value from the event file if available. Fixes #1401 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 21:53:21 +03:00
Sign in to join this conversation.
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#1944
No description provided.