Skip to content

Conversation

@Fatme
Copy link
Contributor

@Fatme Fatme commented Jan 15, 2019

PR Checklist

What is the current behavior?

device list-files, device get-file and device put-file commands fail with an error.

What is the new behavior?

device list-files, device get-file and device put-file commands work correctly.

Fixes/Implements/Closes #[Issue Number].

this.errorInvalidProject(projectDir);
}

public initializeProjectDataSafe(projectDir?: string): void {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a discussion we decided it is better to try/catch when needed, than adding a method.

Copy link
Contributor

@KristianDD KristianDD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After try/catch fix

@Fatme
Copy link
Contributor Author

Fatme commented Jan 17, 2019

run ci

@Fatme Fatme force-pushed the fatme/fix-ios-device-commands branch from 42ddd02 to 1b01435 Compare January 17, 2019 10:05
@Fatme Fatme merged commit 1c04282 into master Jan 17, 2019
@Fatme Fatme deleted the fatme/fix-ios-device-commands branch January 17, 2019 15:09
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.

4 participants