Skip to content

Conversation

lukka
Copy link
Member

@lukka lukka commented May 21, 2025

No description provided.

@lukka lukka requested a review from a team as a code owner May 21, 2025 20:32
@github-project-automation github-project-automation bot moved this to Pull Request in cpptools May 21, 2025
@lukka lukka force-pushed the dev/cpp-copilot-context branch from 5b74d1b to 9c773ee Compare May 21, 2025 20:36
Copy link
Contributor

@sean-mcmanus sean-mcmanus left a comment

Choose a reason for hiding this comment

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

Improved->Improve the.

@lukka lukka force-pushed the dev/cpp-copilot-context branch 2 times, most recently from 1b7b426 to d6b7e9f Compare May 21, 2025 20:38
sean-mcmanus
sean-mcmanus previously approved these changes May 21, 2025
Copy link
Contributor

@sean-mcmanus sean-mcmanus left a comment

Choose a reason for hiding this comment

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

Umm...I'm not sure if the "the" is needed before "context" -- it somewhat depends on the meaning that is intended.

@lukka
Copy link
Member Author

lukka commented May 21, 2025

Umm...I'm not sure if the "the" is needed before "context" -- it somewhat depends on the meaning that is intended.

I agree "the" should be added in this case, let me know.

@lukka lukka merged commit 50da8ef into main May 21, 2025
6 checks passed
@lukka lukka deleted the dev/cpp-copilot-context branch May 21, 2025 20:59
@github-project-automation github-project-automation bot moved this from Pull Request to Done in cpptools May 21, 2025
sean-mcmanus added a commit that referenced this pull request May 29, 2025
* Remove -D__building_module(x)=0 workaround. (#13621)

* add note about C++ Copilot suggestions improvement (#13624)

* Update changelog for 1.26.1 (#13627)

* Update changelog for 1.26.1

* fix typo (#13630)

1.26.0->1.26.1

* Remove the .type.descriptions since they don't seem to have any effect. (#13648)

* Lock the PATH variable which was incorrectly being translated. (#13649)

* Fix a couple loc issues. (#13650)

---------

Co-authored-by: Luca <[email protected]>
Co-authored-by: heartacker <[email protected]>
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.

3 participants