-
-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: --include
parameter for metadata contained in file
#678
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #678 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 32 34 +2
Lines 905 948 +43
=========================================
+ Hits 905 948 +43
☔ View full report in Codecov by Sentry. |
0501f8b
to
9dc07dc
Compare
9dc07dc
to
9d9b4ca
Compare
Code Climate has analyzed commit 9d9b4ca and detected 0 issues on this pull request. View more on Code Climate. |
Shipped in release $ sfdx plugins:install sfdx-git-delta@latest-rc
$ sfdx plugins:install [email protected] |
Explain your changes
Include implementation is now done in post processing.
It deals with inFile metadata more gracefully allowing them to be listed in package.xml and copied (when
-d
is set)include destructive also benefit from this improvement
README file has been updated to explain the specificity of this implementation as it requires full repo fetch depth.
micromatch
has been removed from the dependency and ignore implementation has been reused for include (exact inverted logic)E2E tests have been added for include, include-destructive, ignore and ignore-destructive cases
Does this close any currently open issues?
closes #677
Any particular element that can be tested locally
You can use the issue/677 branch in the reproduction playground