mirror of
https://github.com/nektos/act.git
synced 2026-04-26 01:15:51 +03:00
[PR #1215] [MERGED] fix: keep action inputs #1845
Labels
No labels
area/action
area/cli
area/docs
area/image
area/runner
area/workflow
backlog
confirmed/not-planned
kind/bug
kind/discussion
kind/external
kind/feature-request
kind/question
meta/duplicate
meta/invalid
meta/need-more-info
meta/resolved
meta/wontfix
meta/workaround
needs-work
pull-request
review/not-planned
size/M
size/XL
size/XXL
stale
stale-exempt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/act#1845
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/nektos/act/pull/1215
Author: @KnisterPeter
Created: 6/14/2022
Status: ✅ Merged
Merged: 6/20/2022
Merged by: @cplee
Base:
master← Head:fix/keep-inputs📝 Commits (1)
7263a93fix: keep action inputs📊 Changes
1 file changed (+50 additions, -51 deletions)
View changed files
📝
pkg/runner/step_action_remote.go(+50 -51)📄 Description
Do not initialize inputs a second time (in pre and main step).
When the action setup already run during pre step, we must not
re-run it during the main step, otherwise the inputs will be
overwritten by possible other action inputs.
This is a regression introduced in https://github.com/nektos/act/pull/1204
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.