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 issue #1092 (VSC-1257) #1093

Merged
merged 3 commits into from Dec 12, 2023
Merged

Conversation

dyarkovoy
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed.

Fixes #1092

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Steps to test this pull request

Configure extension with the existing ESP-IDF + installed tools on Windows 11, there should be no errors complaining that the tools are not found at the specified correct paths

How has this been tested?

This has been tested on my VSCode with the existing ESP-IDF + installed tools on Windows 11

Dependent components impacted by this PR:

  • src/idfToolsManager.ts
  • src/utils.ts

Checklist

  • [*] PR Self Reviewed
  • [*] Applied Code formatting
  • [*] Not Added Documentation (irrelevant)
  • [*] Not Added Unit Test (irrelevant)
  • Verified on all platforms - Windows,Linux and macOS (verified only on Windows, as I am not using other OSes at the moment)

@brianignacio5
Copy link
Collaborator

Thank you for your PR. Could you please fix the lint issues so we can merge this changes ?

@github-actions github-actions bot changed the title Fix for issue #1092 Fix for issue #1092 (VSC-1257) Dec 11, 2023
@dyarkovoy
Copy link
Contributor Author

Thank you for your PR. Could you please fix the lint issues so we can merge this changes ?

I believe I just did that by running yarn prettier --write src/*.ts and amended the commit, not sure this is the right way though, can you check?

@brianignacio5 brianignacio5 self-assigned this Dec 12, 2023
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

@brianignacio5 brianignacio5 merged commit 0299537 into espressif:master Dec 12, 2023
6 checks passed
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.

installed ESP-IDF tools are not detected (VSC-1256)
2 participants