Skip to content

feat: flag to limit the thread count used by LS #46

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

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

kittaakos
Copy link
Contributor

@kittaakos kittaakos commented Feb 6, 2024

chore: bump version for the release
test: use the latest CLI version for testing
feat(test): improve logging for tests

  • log when core install starts fix(test): regressions due to platform change
  • change core version to 3.0.0-arduino3r2,
  • the vendor ID is arduino and not esp32 anymore, and
  • use the AVR core for negative tests: it does not have a default programmer set. Hence, the
    expected error is not triggered.

Ref: arduino/arduino-language-server#177


The VSIX is available to download from my fork. Here is the diff against this PR.

 chore: bump version for the release
 test: use the latest CLI version for testing
 feat(test): improve logging for tests
   - log when core install starts
 fix(test): regressions due to platform change
   - change core version to `3.0.0-arduino3r2`,
   - the vendor ID is `arduino` and not `esp32`
anymore, and
   - use the AVR core for negative tests: it does
not have a default programmer set. Hence, the
expected error is no triggered.

Ref: arduino/arduino-language-server#177

Signed-off-by: Akos Kitta <[email protected]>
@kittaakos kittaakos added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Feb 6, 2024
kittaakos pushed a commit to kittaakos/vscode-arduino-tools that referenced this pull request Feb 6, 2024
To be able to download the VSIX from the IDE2 build, it has to be
available somewhere. This commit puts the release-preview VSIX under
version control.

Ref: arduino#46
Signed-off-by: Akos Kitta <[email protected]>
@kittaakos kittaakos merged commit 666dd48 into main Feb 7, 2024
@kittaakos kittaakos deleted the arduino/arduino-language-server#177 branch February 7, 2024 10:19
kittaakos pushed a commit to arduino/arduino-ide that referenced this pull request Feb 7, 2024
Added a new preference (`arduino.language.asyncWorkers`) to control the
number of async workers used by `clangd`.
Users can fine tune the `clangd` thread count to overcome the excessive
CPU usage.

Use 0.1.2 Arduino Tools VSIX in IDE2.

Ref: arduino/arduino-language-server#177
Ref: arduino/vscode-arduino-tools#46

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit to arduino/arduino-ide that referenced this pull request Feb 7, 2024
Added a new preference (`arduino.language.asyncWorkers`) to control the
number of async workers used by `clangd`.
Users can fine tune the `clangd` thread count to overcome the excessive
CPU usage.

Use 0.1.2 Arduino Tools VSIX in IDE2.

Ref: arduino/arduino-language-server#177
Ref: arduino/vscode-arduino-tools#46

Signed-off-by: Akos Kitta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants