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

Fix for getProjectName (VSC-262) #57

Merged
merged 4 commits into from
Mar 18, 2020
Merged

Fix for getProjectName (VSC-262) #57

merged 4 commits into from
Mar 18, 2020

Conversation

pwmb
Copy link
Contributor

@pwmb pwmb commented Mar 11, 2020

getProjectName() was continuing execution even if promise was rejected

Return reject or resolve so that out of branch execution is stopped

@pwmb pwmb added the bugfix Fixing some issue and bug in this PR label Mar 11, 2020
@pwmb pwmb self-assigned this Mar 11, 2020
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot changed the title Fix for getProjectName Fix for getProjectName (VSC-262) Mar 11, 2020
Merge master from espressif/vscode-esp-idf-extension
@pwmb pwmb added this to the v0.1.5 milestone Mar 17, 2020
@pwmb pwmb merged commit e658db1 into espressif:master Mar 18, 2020
brianignacio5 added a commit that referenced this pull request Mar 24, 2020
fix uppercase search (#47)

read Cmakelists file for project name (VSC-248) (#43)

read build project_description json file for project name

Feature/localization kv val (VSC-226) (VSC-246) (#41)

* add gulp i18n validation and src missing key log

rm unused i18n field, add unknown field (#48)

v0.1.4

add gif in readme (VSC-256) (#51)

* add gif in readme

* slower frame rate gif

update idf tools on idf download, refactor some promise await (#54)

Fix for getProjectName (VSC-262) (#57)

* 🐛 Fix continued execution
* Yarn Audit dependency upgrade

Added a custom Debug Adapter

fix old configs remove build before debug

fix debug adapter unit test

DA python pkgs check (#3)

* read Cmakelists file for project name (VSC-248) (#43)

read build project_description json file for project name

* Feature/localization kv val (VSC-226) (VSC-246) (#41)

* add gulp i18n validation and src missing key log

* rm unused i18n field, add unknown field (#48)

* v0.1.4

* Added a custom Debug Adapter

* fix old configs remove build before debug

* removed outdated extension.test.ts

* fix debug adapter unit test

* removed repeated options

* removed redundant requirements.txt

* made `Upload testing.results.log` always executable

* add adapter python reqs checking

Co-authored-by: Soumesh Banerjee <soumeshbanerjee@live.com>
Co-authored-by: Andrey Gramakov <mail@agramakov.me>

Fix/da merge (#4)

* read Cmakelists file for project name (VSC-248) (#43)

read build project_description json file for project name

* Feature/localization kv val (VSC-226) (VSC-246) (#41)

* add gulp i18n validation and src missing key log

* rm unused i18n field, add unknown field (#48)

* v0.1.4

* Added a custom Debug Adapter

* fix old configs remove build before debug

* fix debug adapter unit test

* DA python pkgs check (#3)

* read Cmakelists file for project name (VSC-248) (#43)

read build project_description json file for project name

* Feature/localization kv val (VSC-226) (VSC-246) (#41)

* add gulp i18n validation and src missing key log

* rm unused i18n field, add unknown field (#48)

* v0.1.4

* Added a custom Debug Adapter

* fix old configs remove build before debug

* removed outdated extension.test.ts

* fix debug adapter unit test

* removed repeated options

* removed redundant requirements.txt

* made `Upload testing.results.log` always executable

* add adapter python reqs checking

Co-authored-by: Soumesh Banerjee <soumeshbanerjee@live.com>
Co-authored-by: Andrey Gramakov <mail@agramakov.me>

* fix merge issues

* log adapter output

Co-authored-by: Soumesh Banerjee <soumeshbanerjee@live.com>
Co-authored-by: Andrey Gramakov <mail@agramakov.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixing some issue and bug in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants