Add logging around db config loading by charisk · Pull Request #1875 · github/vscode-codeql

charisk · 2022-12-15T11:31:27Z

In order to avoid flooding the log, we only add log messages only for the initial load of the config.

Checklist

N/A:

CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
Issues have been created for any UI or other user-facing changes made by this pull request.
[Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

shati-patel

⚡

Add logging around db config loading

5e1a136

charisk added the secexp label Dec 15, 2022

charisk requested a review from a team as a code owner December 15, 2022 11:31

shati-patel approved these changes Dec 15, 2022

View reviewed changes

Comment thread extensions/ql-vscode/src/databases/config/db-config-store.ts Outdated

Use 'database' instead of 'db'

feb82db

charisk merged commit 727da3d into main Dec 15, 2022

charisk deleted the charisk/db-config-store-logs branch December 15, 2022 12:21

norascheuch pushed a commit that referenced this pull request Dec 15, 2022

Add logging around db config loading (#1875)

0f4a0b7