Update move and copy commands target path description#115
Update move and copy commands target path description#115eyalbe4 merged 3 commits intojfrog:mainfrom guydelarea:update-target-path-docs
Conversation
…g and adding examples
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
eyalbe4
left a comment
eyalbe4
left a comment
There was a problem hiding this comment.
Thank you for this PR!
See my inline comments.
|
|
||
| #### Example 4 | ||
|
|
||
| Copy all artifacts located under /rabbit in the **source-frog-repo** repository to an absulote path. |
There was a problem hiding this comment.
Since it's a bit hard to understand from the example description what the desired outcome is, I suggest adding a "before" and "after" example that demonstrates the outcome. Also, I'm not sure "to an absolute path" isn't fully clear, so I suggest rephrasing the entire description.
|
|
||
| #### Example 4 | ||
|
|
||
| Move all artifacts located under /rabbit in the **source-frog-repo** repository to an absulote path. |
There was a problem hiding this comment.
Since it's a bit hard to understand from the example description what the desired outcome is, I suggest adding a "before" and "after" example that demonstrates the outcome. Also, I'm not sure "to an absolute path" isn't fully clear, so I suggest rephrasing the entire description.
| Move all artifacts located under /rabbit in the **source-frog-repo** repository to an absulote path. | ||
|
|
||
| ``` | ||
| jf rt cp "source-frog-repo/rabbit/*" target-frog-repo/rabbit/ --flat |
There was a problem hiding this comment.
"cp" here should probably be replaced with "mv".
| #### Example 4 | ||
|
|
||
| Copy all artifacts located under /rabbit in the **source-frog-repo** repository to an absulote path. | ||
| Copy all artifacts located under /rabbit in the **source-frog-repo** repository into the same path in the target-frog-repo repository without maintaining the original subdirectory hierarchy. |
There was a problem hiding this comment.
target-frog-repo --> target-frog-repo
(Notice the markdown ** wrapping)
The same goes for the mv command example.
2 participants
Update the mv and cp commands target path descriptions with the --flat flag, and add examples of moving or copying artifacts to an absolute path.