Change log extension

[benwa]

From .txt to .log

Summary of the Pull Request

Changes the log file extension to .log for enhanced tool usage.

PR Checklist

• Closes: Change log file extension #33800
• Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
• Tests: Added/updated and all pass
• Localization: All end user facing strings can be localized
• Dev docs: Added/updated
• New binaries: Added on the required places
  • JSON for signing for new binaries
  • WXS for installer for new binaries and localization folder
  • YML for CI pipeline for new test projects
  • YML for signed pipeline
• Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Changed all log file extensions from .txt to .log. This honors system and user defined file associations like CMTrace. It also automatically enables syntax highlighting in software like VSCode.

Validation Steps Performed

Manually verified log files and bundled support zip.

[benwa]

@microsoft-github-policy-service agree

[Jay-o-Way]

Is this backwards compatible or a breaking change?

[benwa]

Unless there are automated tools to read log bundles that I don't know about, this should not break anything.

…

On Sun, Jul 14, 2024, 08:26 Jay ***@***.***> wrote: Is this backwards compatible or a breaking change? — Reply to this email directly, view it on GitHub <#33813 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAKJ7E3LUUVV6AZ4XHQ3GGTZMJ4A5AVCNFSM6AAAAABKZZZBHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRXGM2DQOBYHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[benwa]

Is there any interest in this PR?

[crutkas]

Sorry, right now the team was heads down on workspaces and getting stuff prepped for winappsdk upgrade.

Added need review tag for this.

/azp run as it looks like it has test failures, lets see if this was random since i don't think this should have caused that failure

[jaimecbernardo]

Merged in latest main. Note to self: when testing, verify that the tools/ executables take this into account, some of them seem to require using log and we have to verify when testing that they're still picking up the files.