[PR #1078] [CLOSED] fix: programs waiting for input are freezing #1758

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

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/1078
Author: @ChristopherHX
Created: 3/25/2022
Status: Closed

Base: masterHead: containerv2


📝 Commits (10+)

📊 Changes

9 files changed (+149 additions, -75 deletions)

View changed files

📝 cmd/root.go (+2 -0)
📝 pkg/container/docker_run.go (+109 -75)
pkg/container/get_env_list_from_map.go (+11 -0)
📝 pkg/runner/action.go (+1 -0)
📝 pkg/runner/run_context.go (+1 -0)
📝 pkg/runner/runner.go (+1 -0)
📝 pkg/runner/runner_test.go (+10 -0)
📝 pkg/runner/step_docker.go (+1 -0)
pkg/runner/testdata/commands-waiting-for-input-freeze/push.yml (+13 -0)

📄 Description

fix: cancel running commands
feat: more control over tty allocation

Resolves #1072
Resolves #977
Closes #995


🔄 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/1078 **Author:** [@ChristopherHX](https://github.com/ChristopherHX) **Created:** 3/25/2022 **Status:** ❌ Closed **Base:** `master` ← **Head:** `containerv2` --- ### 📝 Commits (10+) - [`401e51c`](https://github.com/nektos/act/commit/401e51c85e2fe679141eca64109ea7690d251f90) fix: programs waiting for input are freezing - [`da06971`](https://github.com/nektos/act/commit/da06971b35c3c0cc70a2e95eef3a48af493efcc8) Silence linter - [`8077f63`](https://github.com/nektos/act/commit/8077f63bf8d7926190c81c179a45141fb467bd0d) Add test for tty freeze - [`1e5463c`](https://github.com/nektos/act/commit/1e5463cd9759eb67724fc887d50fa27317c71e61) Workaround for failed tests - [`b32ab0e`](https://github.com/nektos/act/commit/b32ab0e8174434d88c855b47eafc2cef92a2a81c) fix - [`d606eb5`](https://github.com/nektos/act/commit/d606eb58c2273ce1c3e2d9a372d6d7c9526c39b9) fix - [`6ba1f40`](https://github.com/nektos/act/commit/6ba1f4045adf083c0a2fc6c1bf514e9772de2349) Avoid runner bug - [`f7abb48`](https://github.com/nektos/act/commit/f7abb48df93448778671d7a7f04290c176280b06) Merge - [`4927249`](https://github.com/nektos/act/commit/4927249c0e50eae62b8c6ed457c57cb4b8815979) Move getEnvListFromMap to new file - [`06a7387`](https://github.com/nektos/act/commit/06a7387dc43d5d8333515ac82bcaf5b02e58a1b9) fix: lint start container again, after killing it ### 📊 Changes **9 files changed** (+149 additions, -75 deletions) <details> <summary>View changed files</summary> 📝 `cmd/root.go` (+2 -0) 📝 `pkg/container/docker_run.go` (+109 -75) ➕ `pkg/container/get_env_list_from_map.go` (+11 -0) 📝 `pkg/runner/action.go` (+1 -0) 📝 `pkg/runner/run_context.go` (+1 -0) 📝 `pkg/runner/runner.go` (+1 -0) 📝 `pkg/runner/runner_test.go` (+10 -0) 📝 `pkg/runner/step_docker.go` (+1 -0) ➕ `pkg/runner/testdata/commands-waiting-for-input-freeze/push.yml` (+13 -0) </details> ### 📄 Description fix: cancel running commands feat: more control over tty allocation Resolves #1072 Resolves #977 Closes #995 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 21:52:33 +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#1758
No description provided.