[PR #288] [MERGED] remove .gitignore before docker cp #1403

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

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/288
Author: @o0Ignition0o
Created: 6/21/2020
Status: Merged
Merged: 6/23/2020
Merged by: @cplee

Base: masterHead: igni/remove_gitignore_before_cp


📝 Commits (2)

  • c5c167d Test setup before I try to understand how things work
  • 16c8f94 Remove .gitignore before we run docker cp

📊 Changes

3 files changed (+38 additions, -1 deletions)

View changed files

📝 pkg/runner/runner_test.go (+1 -0)
📝 pkg/runner/step_context.go (+23 -1)
pkg/runner/testdata/issue-228/main.yaml (+14 -0)

📄 Description

Will hopefully fix #228

This patch checks for the presence of a .gitignore file in a remote repository action, and removes it before running docker cp.

I'm not too aware of how the project works and its structure, I'm not even sure that's the way you'd like to address this issue, so please let me know how you feel about it, and feel free to discard it or ask me for edits :)

Thanks again for your time


🔄 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/288 **Author:** [@o0Ignition0o](https://github.com/o0Ignition0o) **Created:** 6/21/2020 **Status:** ✅ Merged **Merged:** 6/23/2020 **Merged by:** [@cplee](https://github.com/cplee) **Base:** `master` ← **Head:** `igni/remove_gitignore_before_cp` --- ### 📝 Commits (2) - [`c5c167d`](https://github.com/nektos/act/commit/c5c167d4d4635a7aee9c12143629ef9203e10806) Test setup before I try to understand how things work - [`16c8f94`](https://github.com/nektos/act/commit/16c8f942dc1309c4ae4d06c83eeddf55aed069e0) Remove .gitignore before we run docker cp ### 📊 Changes **3 files changed** (+38 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `pkg/runner/runner_test.go` (+1 -0) 📝 `pkg/runner/step_context.go` (+23 -1) ➕ `pkg/runner/testdata/issue-228/main.yaml` (+14 -0) </details> ### 📄 Description Will hopefully fix #228 This patch checks for the presence of a `.gitignore` file in a remote repository action, and removes it before running `docker cp`. I'm not too aware of how the project works and its structure, I'm not even sure that's the way you'd like to address this issue, so please let me know how you feel about it, and feel free to discard it or ask me for edits :) Thanks again for your time --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 21:51:02 +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#1403
No description provided.