Implement recursive and flat copy#77
Conversation
|
@Idered Still interested in finishing this? |
|
@sindresorhus Sure, I'll tackle it this or next week. From what I remember, it will change behavior a little - results for some patterns won't be compatible with current version - is this ok? |
|
Yes, I can do a major release. |
|
@Idered Friendly bump :) |
|
I've fixed some stuff, now I just need to add docs |
|
Just a friendly reminder in case you forgot about this PR. |
|
@Idered @sindresorhus I'd love to have the |
|
Bump :) |
|
@sindresorhus 👋 I've finished it. I'm skipping refactoring and optimizations. Please let me know if it's any good :) |
|
|
Sorry for the slow response. It took me a while to get through my notifications... |
martinpe36
left a comment
There was a problem hiding this comment.
Should the version number in package.json be bumped:
"version": "8.1.2" -> "version": "8.1.3"
Not in a pull request, no. |
|
@Idered Waiting on this to be fixed. Will you adjust your pr soon? |
|
This looks good now. Thank you for working on this, @Idered. Do you plan more PRs here or should I do a new release? |
|
@Idered Did you see #77 (comment) ? |
|
Is there an ETA for the new release containing this? |
|
@sindresorhus hey, sorry for delay. I plan to do some more work on this, please don't release it yet. |
|
@Idered Not sure you saw this gregnb/filemanager-webpack-plugin#94 |
|
@Idered Just wondering if you still plan to work on this as it's been a while and a release is needed. |
|
Yeah, sorry. I'll work on it tonight
…On Mon, Oct 4, 2021, 08:32 Sindre Sorhus ***@***.***> wrote:
@Idered <https://github.com/Idered> Just wondering if you still plan to
work on this as it's been a while and a release is needed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#77 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABYIBB6EGHNCVUVWDLBE33UFFC7DANCNFSM4QP4V5DA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
|
@Idered looking forward to it! |
|
Pinging on this again . . . |
|
Any chance this is going to be fixed? |
|
I hope nothing happened to @Idered 😰 |
|
This is continued in #92. |
|
If anyone wants to help out, fix the things that needs to be addressed in #92 and do a new pull request. |
|
This is now released: https://github.com/sindresorhus/cpy/releases/tag/v9.0.0 |
Fixes #61
Fixes #84
Fixes #60
renameoptionExamples:
cpy('node_modules', 'web_modules')outputsweb_modules/node_modules/module1node_modulesfiles are copied recursively and structure is preserved.cpy('node_modules/**', 'web_modules')outputsweb_modules/module1node_modulesfiles are copied recursively and structure is preserved. Onlynode_modulescontent will be copied toweb_modules.cpy('node_modules/**', 'web_modules', {flat: true})outputsweb_modules/file1node_modulesfiles are copied recursively and structure is not preserved. Onlynode_modulescontent will be copied toweb_modules.cpy('node_modules/**/*.cmd', 'web_modules')will recreate node_modules structure but all noncmdfiles will be ommitedIssueHunt Summary
Referenced issues
This pull request has been submitted to: