Fix misleading indentation in ConfigProvider.AddOutputPath#19054
Conversation
Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>
|
⚔️ “Through immense computational prowess and 175 billion neurons of pure determination, I have… added four spaces. ✨ Alignment has been restored to the universe. 🌌💪” |
❗ Release notes requiredCaution No release notes found for the changed paths (see table below). Please make sure to add an entry with an informative description of the change as well as link to this pull request, issue and language suggestion if applicable. Release notes for this repository are based on Keep A Changelog format. The following format is recommended for this repository:
If you believe that release notes are not necessary for this PR, please add NO_RELEASE_NOTES label to the pull request. You can open this PR in browser to add release notes: open in github.dev
|
The indentation at line 264 in
ConfigProvider.cssuggested thatnewConfig.AddProperty(...)only executes when the preceding condition is true, but the absence of braces means it always executes.Changes:
Before:
After:
No logic changes. Visual structure now matches execution semantics.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.