Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 7, 2026

The github-merge-flow.jsonc file had incorrect JSON structure where ResetToTargetPaths was embedded as a command-line argument string within ExtraSwitches rather than as a separate array field.

Changes

  • Restructured each branch configuration to have ResetToTargetPaths as its own JSON array field at the same level as ExtraSwitches
  • Simplified ExtraSwitches to contain only -QuietComments
  • Converted path format from semicolon-separated string to proper JSON array

Before/After

// Before
"release/8.0.1xx": {
    "MergeToBranch": "release/8.0.4xx",
    "ExtraSwitches": "-QuietComments -ResetToTargetPaths global.json;NuGet.config;eng/Version.Details.xml;eng/Versions.props;eng/common/*"
}

// After
"release/8.0.1xx": {
    "MergeToBranch": "release/8.0.4xx",
    "ExtraSwitches": "-QuietComments",
    "ResetToTargetPaths": [
        "global.json",
        "NuGet.config",
        "eng/Version.Details.xml",
        "eng/Versions.props",
        "eng/common/*"
    ]
}

Applied to all 7 branch configurations: release/8.0.1xx, release/8.0.4xx, release/9.0.1xx, release/9.0.3xx, release/10.0.1xx, release/10.0.2xx, and dnup.

Original prompt

The format of https://github.com/dotnet/sdk/blob/main/github-merge-flow.jsonc is incorrect. It looks like ResetToTargetPaths should be it's own node under ExtraSwitches which is a json array object. Can you modify that file to have that format for each entry?

Here's an example I found from another repo.
"ExtraSwitches": "-QuietComments",
"ResetToTargetPaths": [
"eng/Version.Details.xml",
"eng/Version.Details.props"
]


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix format of github-merge-flow.jsonc Fix github-merge-flow.jsonc format: separate ResetToTargetPaths from ExtraSwitches Jan 7, 2026
Copilot AI requested a review from marcpopMSFT January 7, 2026 23:28
@jjonescz
Copy link
Member

jjonescz commented Jan 8, 2026

I have already merged a similar fix: #52344

@marcpopMSFT marcpopMSFT closed this Jan 8, 2026
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.

3 participants