Skip to content

Database population unification#14426

Open
Colengms wants to merge 6 commits intomainfrom
dev/coleng/cpptools_dbpop
Open

Database population unification#14426
Colengms wants to merge 6 commits intomainfrom
dev/coleng/cpptools_dbpop

Conversation

@Colengms
Copy link
Copy Markdown
Contributor

@Colengms Colengms commented May 5, 2026

  • Removed the recursive includes 'reduce' feature, recursive includes would no longer be expanded, so wouldn't need to be reduced.
  • Added some status strings.
  • Changed didChangeCppProperties from a request to a notification.

@Colengms Colengms requested a review from a team as a code owner May 5, 2026 21:11
@github-project-automation github-project-automation Bot moved this to Pull Request in cpptools May 5, 2026
@bobbrow
Copy link
Copy Markdown
Member

bobbrow commented May 5, 2026

Are we removing workspaceParsingPriority as well? Or will we repurpose it or replace it with a setting to control the number of parsing processes?

Copy link
Copy Markdown
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.

Was workspace parsing priority also supposed to be removed or deprecated?

"recursiveIncludes": {
"type": "object",
"properties": {
"reduce": {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be deprecated instead or before it's removed? i.e. internally it would have no effect but users won't get a settings error?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pull Request

Development

Successfully merging this pull request may close these issues.

3 participants