Skip to content

Conversation

@dzsquared
Copy link
Contributor

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (7)
  • SqlScriptDom/Parser/TSql/Ast.xml: Language not supported
  • SqlScriptDom/Parser/TSql/TSql140.g: Language not supported
  • SqlScriptDom/Parser/TSql/TSql150.g: Language not supported
  • SqlScriptDom/Parser/TSql/TSql160.g: Language not supported
  • SqlScriptDom/Parser/TSql/TSql170.g: Language not supported
  • Test/SqlDom/Baselines140/AlterDatabaseOptionsTests140.sql: Language not supported
  • Test/SqlDom/TestScripts/AlterDatabaseOptionsTests140.sql: Language not supported

internal const string Romanian = "ROMANIAN";
internal const string Brazilian = "BRAZILIAN";
internal const string NorwegianBokmal = "NORWEGIAN (BOKMÅL)";
internal const string NorwegianBokmal = "NORWEGIAN (BOKM�L)";
Copy link
Member

Choose a reason for hiding this comment

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

change expected?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope, investigating the git setting that caused this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

file encoding was utf8, which is sensitive to messing up the combined bytes of some characters. the most definitive fix was to convert the file to utf8 bom file encoding, which is used by a number of other c# files in the project already

@dzsquared dzsquared requested review from llali and zijchen January 23, 2025 18:15
@dzsquared dzsquared merged commit 6114856 into main Jan 23, 2025
5 checks passed
@dzsquared dzsquared deleted the dev/fix100 branch January 23, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WAIT_STATS_CAPTURE_MODE not recognized as valid query_store_option_list parameter

4 participants