Skip to content
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

--output-dir #627

Merged
merged 6 commits into from
May 23, 2023
Merged

--output-dir #627

merged 6 commits into from
May 23, 2023

Conversation

WillieRuemmele
Copy link
Contributor

@WillieRuemmele WillieRuemmele commented May 19, 2023

What does this PR do?

adds the --output-dir to project retrieve source
matches --retrievetargetdir from force * retrieve

What issues does this PR fix or reference?

@W-13207990@

see the original PR: salesforcecli/plugin-source#624

@WillieRuemmele WillieRuemmele changed the title Wr/retrieve target dir --output-dir May 19, 2023
package.json Outdated Show resolved Hide resolved
@WillieRuemmele WillieRuemmele force-pushed the wr/retrieveTargetDir branch from 681c1af to 2fdb274 Compare May 23, 2023 19:35
@shetzel
Copy link
Contributor

shetzel commented May 23, 2023

QA with dreamhouse-lwc project

bin/dev project:retrieve:start --source-dir force-app/main/default/classes --output-dir ret1 --> ERROR: --source-dir=force-app/main/default/classes cannot also be provided when using --output-dir
bin/dev project:retrieve:start --package-name foo --output-dir ret1 --> ERROR: --package-name=foo cannot also be provided when using --output-dir
bin/dev project:retrieve:start --metadata ApexClass --output-dir ret1 --> SUCCESS
bin/dev project:retrieve:start --manifest package-classes.xml --output-dir ret2 --> SUCCESS
bin/dev project:retrieve:start --manifest package-classes.xml --output-dir force-app --> ERROR: The retrieve target directory [force-app] overlaps one of your package directories. Specify a different retrieve target directory and try again.

@shetzel shetzel merged commit 40616f0 into main May 23, 2023
@shetzel shetzel deleted the wr/retrieveTargetDir branch May 23, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants