-
Notifications
You must be signed in to change notification settings - Fork 21
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: deploys don't require a project #619
Conversation
@@ -213,7 +173,7 @@ export default class RetrieveMetadata extends SfCommand<RetrieveResultJson> { | |||
const formatter = flags['target-metadata-dir'] | |||
? new MetadataRetrieveResultFormatter(result, { | |||
'target-metadata-dir': flags['target-metadata-dir'], | |||
'zip-file-name': zipFileName, | |||
'zip-file-name': flags['zip-file-name'] ?? DEFAULT_ZIP_FILE_NAME, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why remove the zipFileName
var? duplicating this on L193
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could pass it into buildRetrieveOptions
too?
@@ -140,6 +140,13 @@ describe('end-to-end-test for tracking with an org (single packageDir)', () => { | |||
expect(response?.toDeploy).to.be.an.instanceof(Array).with.lengthOf(0); | |||
}); | |||
|
|||
it('sf deploy no local changes is not an error', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sweet
✅ : resumes deploy in progress from cache
✅ : convert source to mdapi,
❓ : we should improve this output, I'm not sure what's happening
it exited with ❌ : retrieve via package names
^ exits 1 with PDR released version
✅ : requirement for source tracked org removed |
✅ : retrieves with |
What does this PR do?
side effects