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

Change arduino-cli config on switch workspace #1475

Merged
merged 2 commits into from
Oct 19, 2018

Conversation

brusherru
Copy link
Contributor

Previously, when the user switched workspace and then tried to upload something onto the board or install package — arduino-cli works with the old workspace instead of newly selected one.

This PR make arduino-cli works always with the current user's workspace.
When he switches it — it changes arduino-cli .cli-config.yml.

@brusherru brusherru self-assigned this Oct 16, 2018
@brusherru brusherru requested a review from a team October 16, 2018 17:16
Copy link
Contributor

@evgenykochetkov evgenykochetkov left a comment

Choose a reason for hiding this comment

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

Tested on macOS, works as intended👍🏻
The code also looks good.

Copy link
Member

@nkrkv nkrkv left a comment

Choose a reason for hiding this comment

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

Almost LGTM.

A small note below. And should we include it in 0.25.1. It’s a bug fix, so why not?

*
* :: ArduinoCli -> Path -> Promise Object Error
*/
export const updateWorkspace = async (cli, newWsPath) => {
Copy link
Member

Choose a reason for hiding this comment

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

A possibly better name to match terminology: switchWorkspace

@brusherru brusherru force-pushed the refactor-workspace-path-handling branch from b18b65c to 46a55e9 Compare October 19, 2018 09:28
@brusherru brusherru changed the base branch from master to 0.25.x October 19, 2018 09:32
@brusherru brusherru merged commit 1a83415 into 0.25.x Oct 19, 2018
@brusherru brusherru deleted the refactor-workspace-path-handling branch October 19, 2018 09:41
@evgenykochetkov evgenykochetkov added the hotfix Issue/PR for a patch release label Oct 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix Issue/PR for a patch release s:client t:bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants