Skip to content

Enable string length encoding fix in cpptools #13769

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 3 commits into from
Jul 29, 2025

Conversation

Colengms
Copy link
Contributor

There is related native side PR as well.

Switches to our own version of didOpen in order to include an encoding var which was added as part of: microsoft/vscode#209503

@Colengms Colengms marked this pull request as ready for review July 28, 2025 21:06
@Colengms Colengms requested a review from a team as a code owner July 28, 2025 21:06
@Colengms Colengms merged commit 79fe240 into main Jul 29, 2025
6 checks passed
@Colengms Colengms deleted the coleng/cpptools_fix_string_literals_lengths branch July 29, 2025 18:52
@github-project-automation github-project-automation bot moved this from Pull Request to Done in cpptools Jul 29, 2025
sean-mcmanus added a commit that referenced this pull request Aug 7, 2025
* Fix the description of debugServerPath (#13778)

This mentions the non-existent miDebugServerAddress, but the
correct name is actually miDebuggerServerAddress.

* Enable string length encoding fix in cpptools (#13769)

* Try to fix the Windows builds. (#13788)

* Update IntelliSense loc strings. (#13793)

* Makes remote attach picker respect the pipeTransport.quoteArgs config… (#13794)

* Makes remote attach picker respect the pipeTransport.quoteArgs configuration
* fixes linter error - don't compare boolean value to a boolean

* Remove "exceptions" from quoteArgs. (#13796)

* Fix loc for the miDebuggerServerAddress change. (#13797)

* Update changelog for 1.27.0 (2nd time). (#13795)

* Update changelog for 1.27.0 (2nd time).

* Update form-data. (#13800)

* Update form-data.

* Revert didOpen changes in favor of adding encoding to didChangeVisibleTextEditors (#13802)

* fixing formatting (#13810)

* Enable CG trigger on insiders branch

* Handle .txx/tpp headers. (#13811)

* Handle .txx headers.

* Add tpp too.

* fix #13818 (#13824)

* Bump tmp from 0.2.3 to 0.2.4 in /Extension (#13825)

Bumps [tmp](https://github.com/raszi/node-tmp) from 0.2.3 to 0.2.4.
- [Changelog](https://github.com/raszi/node-tmp/blob/master/CHANGELOG.md)
- [Commits](raszi/node-tmp@v0.2.3...v0.2.4)

---
updated-dependencies:
- dependency-name: tmp
  dependency-version: 0.2.4
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sean McManus <seanmcm@microsoft.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Joshua Goins <josh@redstrate.com>
Co-authored-by: Colen Garoutte-Carson <49173979+Colengms@users.noreply.github.com>
Co-authored-by: Matt <59707001+mjrist@users.noreply.github.com>
Co-authored-by: Bob Brown <bobbrow@users.noreply.github.com>
Co-authored-by: Luca <681992+lukka@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants