Skip to content

Conversation

naveenkaje
Copy link
Contributor

Description

The zipped project should include the .vscode directory. This is a fix for #8411

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@naveenkaje
Copy link
Contributor Author

Behavior with this change
capture

@naveenkaje
Copy link
Contributor Author

@theotherjimmy @bridadan Please review.

@ciarmcom ciarmcom requested review from a team March 6, 2019 00:00
@ciarmcom
Copy link
Member

ciarmcom commented Mar 6, 2019

@naveenkaje, thank you for your changes.
@ARMmbed/mbed-os-tools @ARMmbed/mbed-os-maintainers please review.

The zipped project should include the .vscode direcotry.
@bridadan
Copy link
Contributor

bridadan commented Mar 6, 2019

@naveenkaje I've been diving deep down the rabbit hole on this issue trying to figure out why this was an issue in the first place. Turns out there was a bigger issue that this was stemming from. May I suggest closing this one as I'd like to get to the root cause?

@naveenkaje
Copy link
Contributor Author

#9967
(27fc19f8e9df19956e60f362680a70b3ee61afc1) has been submitted to address this issue. Hence closing this P.R

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.

5 participants