appleboy / appleboy/scp-action

Don' t remove the target folder with option "rm", empty it instead

Open
#43 4 comments 9 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
1.6k
Forks
178
PR merge metrics
No merged PRs in 30d

Description

I don't think to remove the target folder and then recreate it is a good idea, consider empty the sub-files and sub-folders in the target folder.

I think this could avoid some problems like docker's mounting

Contributor guide

No contributing guide indexed for this repository

Research direction

No file or test is named. Start by locating the implementation of the rm option and trace how it handles the target folder; confirm the current behavior and how Docker mounting is affected. Done means the target folder remains in place while its contents are emptied, with coverage for the changed behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, shell
Domain
ci-cd, devops
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.