Skip to content

Rename db config terms to use 'variant analysis' instead of 'remote'#1960

Merged
charisk merged 1 commit intomainfrom
charisk/rename-remote-in-db-config
Jan 13, 2023
Merged

Rename db config terms to use 'variant analysis' instead of 'remote'#1960
charisk merged 1 commit intomainfrom
charisk/rename-remote-in-db-config

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Jan 12, 2023

Details in linked internal issue.

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.

@charisk charisk added the secexp label Jan 12, 2023
@charisk charisk requested review from a team as code owners January 12, 2023 16:47
Copy link
Copy Markdown
Contributor

@norascheuch norascheuch left a comment

Choose a reason for hiding this comment

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

Changes look good! (even if I don't understand the new name, especially in relation to local databases)

@charisk charisk merged commit 78e01e6 into main Jan 13, 2023
@charisk charisk deleted the charisk/rename-remote-in-db-config branch January 13, 2023 09:30
@charisk charisk mentioned this pull request Jan 18, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants